]> WPIA git - gigi.git/commitdiff
chg: ignore NoSuchFileException for Pwned Passwords
authorLucas Werkmeister <mail@lucaswerkmeister.de>
Tue, 19 Jun 2018 21:20:22 +0000 (23:20 +0200)
committerLucas Werkmeister <mail@lucaswerkmeister.de>
Tue, 19 Jun 2018 21:21:33 +0000 (23:21 +0200)
If we can’t open the Pwned Passwords database because the file does not
exist, there’s no need to print a detailed stack trace: the warning
message should be enough to gently inform the system administrator that
they can improve their security by installing the database. Any other
errors (e. g. permission errors) are still reported.

This is mainly motivated by the dozens of NoSuchFileException stack
traces in CI builds, which this commit should silence.

Change-Id: Id08afc1600a70acfc49b2c4335b533949413b09a

src/club/wpia/gigi/Gigi.java

index 660c3d308c9645d7f3003b7cee51d1cb8115c132..36e8c8ccc1064f17486dc7ce4e086d66ccc5d2fa 100644 (file)
@@ -6,6 +6,7 @@ import java.io.UnsupportedEncodingException;
 import java.math.BigInteger;
 import java.nio.channels.FileChannel;
 import java.nio.file.FileSystems;
+import java.nio.file.NoSuchFileException;
 import java.nio.file.Path;
 import java.security.KeyStore;
 import java.security.MessageDigest;
@@ -319,7 +320,9 @@ public final class Gigi extends HttpServlet {
                 throw new RuntimeException("Error while opening password hash database, refusing startup", e);
             } else {
                 System.err.println("Error while opening password hash database, passwords will be checked only by strength");
-                e.printStackTrace();
+                if (!(e instanceof NoSuchFileException)) {
+                    e.printStackTrace();
+                }
                 return new PasswordStrengthChecker();
             }
         }