Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[UNDERTOW-466] Split the SecurityContextImpl in two. #317

Merged
merged 1 commit into from
Jun 4, 2015

Conversation

darranl
Copy link
Contributor

@darranl darranl commented Jun 4, 2015

Split out the generic notification handling side of the SecurityContext implementation into an abstract base class so future SecurityContext implementations can focus just on the actual authentication.

Note: I have chosen not to add the deprecated methods to the abstract class as I believe once Java 8 is available it will be better to handle them on the interface to make implementing those methods entirely optional.

…e SecurityContext implementation into an abstract base class so future SecurityContext implementations can focus just on the actual authentication.
@undertow-pull-request
Copy link

Windows Build 1191 is now running using a merge of fe2b72e

@undertow-pull-request
Copy link

Linux Build 1471 is now running using a merge of fe2b72e

@undertow-pull-request
Copy link

Windows Build 1191 outcome was FAILURE using a merge of fe2b72e
Summary: Tests failed: 4 (2 new), passed: 2503, ignored: 375 Build time: 0:13:26

Build problems:

Failed tests detected

Failed tests

io.undertow.websockets.core.protocol.WebSocket08ServerTest.testCloseFrame{proxy}: <no details avaliable>

io.undertow.server.handlers.proxy.LoadBalancingProxyAJPTestCase.testStickySessions{proxy}{spdy}: <no details avaliable>

io.undertow.server.security.SpnegoDigestAuthenticationTestCase.testDigestSuccess: <no details avaliable>

io.undertow.server.security.SpnegoDigestAuthenticationTestCase.testDigestSuccess{proxy}{http2-clear}: <no details avaliable>

@undertow-pull-request
Copy link

Linux Build 1471 outcome was FAILURE using a merge of fe2b72e
Summary: Tests failed: 2 (2 new), passed: 2582, ignored: 298 Build time: 0:15:25

Build problems:

Failed tests detected

Failed tests

io.undertow.server.handlers.proxy.mod_cluster.StickySessionForceUnitTestCase.testDomainNodeInError{proxy}{https}: <no details avaliable>

io.undertow.server.handlers.encoding.EncodingSelectionTestCase.testEncodingSelectWithQValue{proxy}{https}: <no details avaliable>

stuartwdouglas added a commit that referenced this pull request Jun 4, 2015
[UNDERTOW-466] Split the SecurityContextImpl in two.
@stuartwdouglas stuartwdouglas merged commit c44f777 into undertow-io:master Jun 4, 2015
@darranl darranl deleted the UNDERTOW-466 branch June 4, 2015 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants