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

[stable14] suppress wrong audit log messages about failed login attempts #12372

Merged
merged 1 commit into from Nov 13, 2018

Conversation

Projects
None yet
3 participants
@schiessle
Member

schiessle commented Nov 9, 2018

backport of #12148

first check if the user is already logged in and then try to authenti…
…cate via apache, this way we suppress wrong audit log messages about failed login attempts

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

@schiessle schiessle added this to the Nextcloud 14.0.4 milestone Nov 9, 2018

@schiessle schiessle requested review from rullzer and MorrisJobke Nov 9, 2018

@MorrisJobke

Code makes sense 👍

@MorrisJobke MorrisJobke referenced this pull request Nov 13, 2018

Merged

14.0.4 RC 1 #12436

@MorrisJobke MorrisJobke merged commit fb559ed into stable14 Nov 13, 2018

1 check failed

continuous-integration/drone/pr the build failed
Details

@MorrisJobke MorrisJobke deleted the suppress-wrong-audit-log-message-stable14 branch Nov 13, 2018

@MorrisJobke MorrisJobke referenced this pull request Nov 22, 2018

Merged

14.0.4 #12586

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment