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

Forbid empty passwords when using LDAP #2283

Merged
merged 2 commits into from May 26, 2016

Conversation

Projects
None yet
2 participants
@joschi
Contributor

joschi commented May 25, 2016

Closes #2214

@joschi joschi added this to the 2.0.2 milestone May 25, 2016

@joschi

This comment has been minimized.

Contributor

joschi commented May 25, 2016

This PR should also be merged into the master branch.

@dennisoelkers dennisoelkers self-assigned this May 25, 2016

@dennisoelkers dennisoelkers merged commit 67b5447 into 2.0 May 26, 2016

4 checks passed

ci-server-integration Jenkins build graylog2-server-integration-pr 920 has succeeded
Details
ci-web-linter Jenkins build graylog-pr-linter-check 408 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@dennisoelkers dennisoelkers deleted the issue-2214 branch May 26, 2016

dennisoelkers added a commit that referenced this pull request May 26, 2016

Forbid empty passwords when using LDAP (#2283)
* Forbid empty passwords when using LDAP

Closes #2214

* Forbid LDAP bind with empty principal or credentials

(cherry picked from commit 67b5447)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment