Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
gplazma: configured banfile plugin should ignore non-existent ban file
Motivation: When configured, gplazma requires a banfile to be present, even if it is empty. When one does not exist, gplazma will report the following on initialization, but continue to start: `failed to create banfile: config file doesn't exist or not a file`. All subsequent login requests will fail and result in a NullPointerException. Modification: As a workaround, the banfile plugin, even if configured, will now ignore an inexistent ban file and not fail with a NullPointerException on every subsequent login attempt. When a banfile is added or filled at some point, it will be used for subseqent login attempts. Eventually, failed plugins should be ignored by gplazma, or gplazma should fail when trying to load them. Result: Even if configured, an empty or inexistent banfile will be ignored and logins should succeed. Target: master, 9.2, 9.1, 9.0, 8.2 Fixes: #7465 Requires-notes: yes Requires-book: no Acked-by: Dmitry Litvintsev, Paul Millar, Tigran Mkrtchyan, Marina Sahakyan
- Loading branch information
Showing
3 changed files
with
15 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters