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

Refactor authentication for servlet HTTP header handler #980

merged 3 commits into from Dec 10, 2015


Copy link

@mrjoel mrjoel commented Dec 9, 2015

This is an update of #273, nearly identical except updated to work with #908 as indicated in the original.

@mrjoel mrjoel force-pushed the mrjoel-httpheaders branch 2 times, most recently from a395b2d to aed0aa3 Compare Dec 9, 2015
mrjoel added 3 commits Dec 9, 2015
This was unused and causing provider lookup to fail in
AuthenticationManager.findProvider() by changing it out
from underneath. As a result, the supportXChanges methods
weren't being reported correctly.
Copy link

gitblit commented Dec 10, 2015

Nice & clean. Great job.

gitblit added a commit that referenced this pull request Dec 10, 2015
Refactor authentication for servlet HTTP header handler
@gitblit gitblit merged commit 7b7b0d5 into gitblit-org:master Dec 10, 2015
Copy link

gitblit commented Dec 10, 2015

I granted you contributor access for easier branch-based PRs rather than from forks. Thanks for all your contributions.

@mrjoel mrjoel deleted the mrjoel-httpheaders branch Dec 10, 2015
@paulsputer paulsputer mentioned this pull request Apr 14, 2016
9 tasks
@flaix flaix modified the milestone: 1.8.0 Mar 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

3 participants