about summary refs log tree commit diff
path: root/web/panettone
diff options
context:
space:
mode:
authorsterni <sternenseemann@systemli.org>2021-04-09T23·51+0200
committersterni <sternenseemann@systemli.org>2021-04-10T09·18+0000
commit055fe7ce8eea844cbdd28464e6d5932e2c84af5f (patch)
tree9e994f5636ec215b049faeea7b9a8365b9571e18 /web/panettone
parentce4476360f0aab1bd542292bf420145996d47f58 (diff)
fix(panettone): always use displayname in subjects r/2459
Emails for (re)opening and closing where send out with the user's DN in
the subject which is probably not what we want.

Using displayname-if-known is probably not necessary as there is not
really a case where (not *user*) wouldn't justify a 500 in this context.

Change-Id: Id12d3d9619f42eb5337c2d3482b7c1646b5d6a81
Reviewed-on: https://cl.tvl.fyi/c/depot/+/2911
Tested-by: BuildkiteCI
Reviewed-by: tazjin <mail@tazj.in>
Diffstat (limited to 'web/panettone')
-rw-r--r--web/panettone/src/panettone.lisp4
1 files changed, 2 insertions, 2 deletions
diff --git a/web/panettone/src/panettone.lisp b/web/panettone/src/panettone.lisp
index 2f15dbff0a..3d803e1fb6 100644
--- a/web/panettone/src/panettone.lisp
+++ b/web/panettone/src/panettone.lisp
@@ -592,7 +592,7 @@ given subject an body (in a thread, to avoid blocking)"
      :subject (format nil "b/~A: \"~A\" closed by ~A"
                       id
                       (subject issue)
-                      (dn *user*))
+                      (displayname *user*))
      :message (link-to-issue id)))
   (hunchentoot:redirect (format nil "/issues/~A" id)))
 
@@ -616,7 +616,7 @@ given subject an body (in a thread, to avoid blocking)"
      :subject (format nil "b/~A: \"~A\" reopened by ~A"
                       id
                       (subject issue)
-                      (dn *user*))
+                      (displayname *user*))
      :message (link-to-issue id)))
   (hunchentoot:redirect (format nil "/issues/~A" id)))