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.

Sign up for GitHub

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

Jump to bottom

Fixes #17998 - Only print LDAP bind status if available #4167

Merged
merged 1 commit into from
Jan 10, 2017
Merged

Fixes #17998 - Only print LDAP bind status if available #4167

merged 1 commit into from
Jan 10, 2017

Commits on Jan 10, 2017

  1. Fixes #17998 - Only print LDAP bind status if available 

    The ldap logger tries to call payload[:bind].status but it might not be
defined if the bind goes wrong in some LDAP sources.

This causes a 500 instead of logging the status of the bind. We should
only display payload[:bind].status if it's available.
    @dLobatog
    dLobatog committed Jan 10, 2017
    Configuration menu
    Copy the full SHA
    09a9a69 View commit details
    Browse the repository at this point in the history