about summary refs log tree commit diff
diff options
context:
space:
mode:
authorGriffin Smith <grfn@gws.fyi>2020-07-26T21·42-0400
committerglittershark <grfn@gws.fyi>2020-07-26T21·44+0000
commit273053dbf3287c02dda60b442cf138a412d8f75b (patch)
treefed312d8fadeae94529b3232d00002317075dca5
parente231493ff4207e1fe3191ba13ece8c7477c3f242 (diff)
fix(web/panettone): Don't log backtraces r/1484
The default hunchentoot behavior is to log all local variables when
logging lisp backtraces - this is nice for debugging, but means that if
we hit an error when checking for auth with the ldap server we log the
password provided by the user. No good! Let's just turn off logging of
backtraces for now.

Change-Id: Ibc4242e3e0f974ac53fffc482d3724b0547425ab
Reviewed-on: https://cl.tvl.fyi/c/depot/+/1471
Reviewed-by: glittershark <grfn@gws.fyi>
Tested-by: BuildkiteCI
-rw-r--r--web/panettone/src/panettone.lisp2
1 files changed, 2 insertions, 0 deletions
diff --git a/web/panettone/src/panettone.lisp b/web/panettone/src/panettone.lisp
index c84a206811..7d29edb7ac 100644
--- a/web/panettone/src/panettone.lisp
+++ b/web/panettone/src/panettone.lisp
@@ -547,6 +547,8 @@ updated issue"
   (let ((port (integer-env "PANETTONE_PORT" :default 6161))
         (ldap-port (integer-env "LDAP_PORT" :default 389))
         (data-dir (or (uiop:getenvp "PANETTONE_DATA_DIR") "/var/lib/panettone")))
+    (setq hunchentoot:*show-lisp-backtraces-p* nil)
+    (setq hunchentoot:*log-lisp-backtraces-p* nil)
     (start-panettone :port port
                      :data-dir data-dir
                      :ldap-port ldap-port)