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

improve log output when no LDAP user was found on login attempt #22880

Merged
merged 1 commit into from
Mar 7, 2016

Conversation

blizzz
Copy link
Contributor

@blizzz blizzz commented Mar 4, 2016

Hopefully eases #21656

To the exception message hostname and port are appended, like this:

No user available for the given login name on ldap://172.116.117.16:389

@davitol @nickvergessen @MorrisJobke are you okay with this refinement?

@blizzz blizzz added this to the 9.1-next milestone Mar 4, 2016
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @MorrisJobke, @LukasReschke and @Xenopathic to be potential reviewers

@nickvergessen
Copy link
Contributor

👍

1 similar comment
@MorrisJobke
Copy link
Contributor

👍

DeepDiver1975 added a commit that referenced this pull request Mar 7, 2016
improve log output when no LDAP user was found on login attempt
@DeepDiver1975 DeepDiver1975 merged commit 0487b54 into master Mar 7, 2016
@DeepDiver1975 DeepDiver1975 deleted the fix-21656 branch March 7, 2016 16:12
@lock
Copy link

lock bot commented Aug 7, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants