<pre style='margin:0'>
Rainer Müller (raimue) pushed a commit to branch master
in repository trac.macports.org.

</pre>
<p><a href="https://github.com/macports/trac.macports.org/commit/3e187627252f55fe78a3cb8f1a94f002237a7f4d">https://github.com/macports/trac.macports.org/commit/3e187627252f55fe78a3cb8f1a94f002237a7f4d</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit 3e187627252f55fe78a3cb8f1a94f002237a7f4d
</span>Author: Rainer Müller <raimue@macports.org>
AuthorDate: Sat Jan 26 15:53:04 2019 +0000

<span style='display:block; white-space:pre;color:#404040;'>    trac-github-update: Remove [bot] tag from email address
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    GitHub Apps usernames contain a [bot] tag which is also supplied in the email
</span><span style='display:block; white-space:pre;color:#404040;'>    address in the WebHook payload. Such an email address would be invalid and
</span><span style='display:block; white-space:pre;color:#404040;'>    resulted in garbage in the From: header, therefore avoid this altogether.
</span>---
 plugins/hooks/trac-github-update.py | 9 +++++++++
 1 file changed, 9 insertions(+)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/plugins/hooks/trac-github-update.py b/plugins/hooks/trac-github-update.py
</span><span style='display:block; white-space:pre;color:#808080;'>index 3203582..3ee7b8b 100755
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/plugins/hooks/trac-github-update.py
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/plugins/hooks/trac-github-update.py
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -120,6 +120,15 @@ class GitHubAPI(object):
</span>         email = knownemail
         if not email:
             email = "%s@users.noreply.github.com" % (username,)
<span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        # Apply a workaround for GitHub Apps that supplies usernames ending
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        # with a [bot] tag that is rendered as a label on the GitHub web
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        # interface. The WebHook also keeps this [bot] in the local part of the
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        # email address, which is in fact invalid and produces garbage in
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        # the From: header.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        if email and email.endswith("[bot]@users.noreply.github.com"):
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            email = "%s@users.noreply.github.com" % (username[:-5],)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span>         if name and email:
             self.cache[username] = (name, email)
             return self.cache[username]
</pre><pre style='margin:0'>

</pre>