diff options
| author | Mark Sapiro <mark@msapiro.net> | 2015-06-23 12:53:50 -0700 |
|---|---|---|
| committer | Mark Sapiro <mark@msapiro.net> | 2015-06-23 12:53:50 -0700 |
| commit | 5c01d482cc37706251892ea1b620b221da2d2ca4 (patch) | |
| tree | 7138ce95a77309550e3657e8e9656f55422a99ab /Mailman/Cgi/subscribe.py | |
| parent | ab19a1505dd93eca5d9ca6792740c9eb56302cfe (diff) | |
| parent | 28f5f0ce0be5529598124bbe5e0d72b0fd605e69 (diff) | |
| download | mailman2-5c01d482cc37706251892ea1b620b221da2d2ca4.tar.gz mailman2-5c01d482cc37706251892ea1b620b221da2d2ca4.tar.xz mailman2-5c01d482cc37706251892ea1b620b221da2d2ca4.zip | |
Improved identification of remote clients coming via a proxy server.
Diffstat (limited to 'Mailman/Cgi/subscribe.py')
| -rwxr-xr-x | Mailman/Cgi/subscribe.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/Mailman/Cgi/subscribe.py b/Mailman/Cgi/subscribe.py index fff21e98..ab5c7cd8 100755 --- a/Mailman/Cgi/subscribe.py +++ b/Mailman/Cgi/subscribe.py @@ -118,9 +118,10 @@ def process_form(mlist, doc, cgidata, lang): # Canonicalize the full name fullname = Utils.canonstr(fullname, lang) # Who was doing the subscribing? - remote = os.environ.get('REMOTE_HOST', - os.environ.get('REMOTE_ADDR', - 'unidentified origin')) + remote = os.environ.get('HTTP_FORWARDED_FOR', + os.environ.get('HTTP_X_FORWARDED_FOR', + os.environ.get('REMOTE_ADDR', + 'unidentified origin'))) # Are we checking the hidden data? if mm_cfg.SUBSCRIBE_FORM_SECRET: now = int(time.time()) |