Unified Diff: sitescripts/submit_email/web/submit_email.py
Issue 29323033:
Issue 2846 - Service for processing Microsoft Edge announcement subscription (Closed)
Patch Set: Default to adblockbrowser
Created Aug. 19, 2015, 1:49 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« sitescripts/submit_email/template/edge_verification.mail ('k')
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: sitescripts/submit_email/web/submit_email.py |
| =================================================================== |
| --- a/sitescripts/submit_email/web/submit_email.py |
| +++ b/sitescripts/submit_email/web/submit_email.py |
| @@ -49,8 +49,12 @@ |
| if lang: |
| params.append(('lang', lang)) |
| + product = data.get('product', 'adblockbrowser') |
|
Sebastian Noack
2015/08/19 15:02:47
Nit: Since we use the same default below, mind sto
Oleksandr
2015/08/19 15:11:24
Done.
|
| + email_template = product + '_verification_email_template' |
| + params.append(('product', product)) |
| + |
| sendMail( |
| - config.get('submit_email', 'verification_email_template'), |
| + config.get('submit_email', email_template), |
| { |
| 'recipient': email, |
| 'verification_url': '%s?%s' % ( |
| @@ -79,7 +83,9 @@ |
| 'Invalid signature in verification request.' |
| ) |
| - filename = config.get('submit_email', 'filename') |
| + product = params.get('product', 'adblockbrowser') |
| + filename = config.get('submit_email', product + '_filename') |
| + |
| with open(filename, 'ab', 0) as file: |
| fcntl.lockf(file, fcntl.LOCK_EX) |
| try: |
« no previous file with comments
|
« sitescripts/submit_email/template/edge_verification.mail ('k')
|
no next file »
|
no next file with comments »
