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

Allow using colon character (":") in password #4557

Merged
merged 1 commit into from Feb 2, 2018
Merged

Allow using colon character (":") in password #4557

merged 1 commit into from Feb 2, 2018

Conversation

@joschi
Copy link
Contributor

@joschi joschi commented Feb 2, 2018

ShiroSecurityContextFilter incorrectly parsed the HTTP Basic Auth header and
prevented users from using the colon character (":") in their passwords.

Refs #4547
(cherry picked from commit 6a49304)

`ShiroSecurityContextFilter` incorrectly parsed the HTTP Basic Auth header and
prevented users from using the colon character (":") in their passwords.

Refs #4547
(cherry picked from commit 6a49304)
@joschi joschi added this to the 2.4.4 milestone Feb 2, 2018
@ghost ghost assigned joschi Feb 2, 2018
@bernd
bernd approved these changes Feb 2, 2018
@bernd bernd merged commit 0455335 into 2.4 Feb 2, 2018
3 of 5 checks passed
3 of 5 checks passed
ci-web-linter Jenkins build graylog-pr-linter-check 2239 has failed
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
graylog-project/pr Jenkins build graylog-project-pr-snapshot 982 has succeeded
Details
license/cla Contributor License Agreement is signed.
Details
@ghost ghost removed the ready-for-review label Feb 2, 2018
@bernd bernd self-assigned this Feb 2, 2018
@bernd bernd deleted the pr-4547-2.4 branch Feb 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants