Log failed accesses #586

Closed
opened 2026-02-04 21:20:14 +03:00 by OVERLORD · 8 comments
Owner

Originally created by @benrubson on GitHub (Mar 2, 2018).

Hello,

I think it would be really nice for BookStack to log failed authentications.
We would then be able to monitor these messages and ban malicious IPs, at firewall level, with for example Fail2Ban on a Linux server.

An easy and nice solution would simply be to log a message into the webserver log for each failed authentication attempt or misbehaviour, using for example PHP error_log() function.

Thank you very much 👍

Ben

Originally created by @benrubson on GitHub (Mar 2, 2018). Hello, I think it would be really nice for BookStack to log failed authentications. We would then be able to monitor these messages and ban malicious IPs, at firewall level, with for example Fail2Ban on a Linux server. An easy and nice solution would simply be to log a message into the webserver log for each failed authentication attempt or misbehaviour, using for example PHP `error_log()` function. Thank you very much 👍 Ben
Author
Owner

@ssddanbrown commented on GitHub (Mar 12, 2018):

Thanks for this suggestion @benrubson, I really like this idea. Will really provide some useful security insight.

@ssddanbrown commented on GitHub (Mar 12, 2018): Thanks for this suggestion @benrubson, I really like this idea. Will really provide some useful security insight.
Author
Owner

@917huB commented on GitHub (Apr 15, 2018):

+1, adding this logging would be appreciated.

@917huB commented on GitHub (Apr 15, 2018): +1, adding this logging would be appreciated.
Author
Owner

@berez23 commented on GitHub (May 8, 2018):

+1

@berez23 commented on GitHub (May 8, 2018): +1
Author
Owner

@ssddanbrown commented on GitHub (May 12, 2018):

@917huB @berez23 Please could you thumbs-up the original issue post to show support as we can sort issues by this meaning your support for this issue will be take more into account when we are deciding on new features to implement.

@ssddanbrown commented on GitHub (May 12, 2018): @917huB @berez23 Please could you thumbs-up the original issue post to show support as we can sort issues by this meaning your support for this issue will be take more into account when we are deciding on new features to implement.
Author
Owner

@917huB commented on GitHub (May 12, 2018):

done thumbed-up :) 👍 thank you.

@917huB commented on GitHub (May 12, 2018): done thumbed-up :) 👍 thank you.
Author
Owner

@mvlnetdev commented on GitHub (Jul 18, 2018):

Thumbed up too! Would be amazing.

@mvlnetdev commented on GitHub (Jul 18, 2018): Thumbed up too! Would be amazing.
Author
Owner

@benrubson commented on GitHub (Feb 8, 2020):

See PR #1881 which solves this 👍

@benrubson commented on GitHub (Feb 8, 2020): See PR #1881 which solves this 👍
Author
Owner

@ssddanbrown commented on GitHub (Jul 28, 2020):

Thanks to @benrubson this will be part of the future v0.30 release.

@ssddanbrown commented on GitHub (Jul 28, 2020): Thanks to @benrubson this will be part of the future v0.30 release.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#586