From 48880b34534ce0ce571c7e58204de3f5ab252943 Mon Sep 17 00:00:00 2001 From: Todd Zullinger Date: Sun, 22 Nov 2009 12:54:05 -0500 Subject: [PATCH 3/4] upload.cgi: Use pkg as the name of the first argument of send_email There are too many *name variables in this function already, so make it clearer that the first is the package name rather than a user or file name. --- configs/web/applications/upload.cgi | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configs/web/applications/upload.cgi b/configs/web/applications/upload.cgi index 727deea..98ed92f 100644 --- a/configs/web/applications/upload.cgi +++ b/configs/web/applications/upload.cgi @@ -61,8 +61,8 @@ def check_auth(username): pass return authenticated -def send_email(name, md5, filename, username): - text = """A file has been added to the lookaside cache for %(name)s: +def send_email(pkg, md5, filename, username): + text = """A file has been added to the lookaside cache for %(pkg)s: %(md5)s %(filename)s""" % locals() msg = MIMEText(text) @@ -81,7 +81,7 @@ def send_email(name, md5, filename, username): sender_name = Header.Header(sender_name, 'utf-8').encode() msg.set_charset('utf-8') sender = Utils.formataddr((sender_name, sender_email)) - recipients = [ '%s-owner@fedoraproject.org' % name, \ + recipients = [ '%s-owner@fedoraproject.org' % pkg, \ 'fedora-extras-commits@redhat.com' ] msg['Subject'] = 'File %s uploaded to lookaside cache by %s' % ( filename, username) msg['From'] = sender -- 1.6.5.2