Merge lp:~stefanor/ibid/passwords-723405 into lp:ibid

Proposed by Stefano Rivera
Status: Merged
Approved by: Max Rabkin
Approved revision: 1038
Merged at revision: 1040
Proposed branch: lp:~stefanor/ibid/passwords-723405
Merge into: lp:ibid
Diff against target: 12 lines (+1/-1)
1 file modified
ibid/plugins/codecontest.py (+1/-1)
To merge this branch: bzr merge lp:~stefanor/ibid/passwords-723405
Reviewer Review Type Date Requested Status
Max Rabkin Approve
Keegan Carruthers-Smith Approve
Review via email: mp+80762@code.launchpad.net

Commit message

Allow spaces in USACO passwords

Description of the change

Untested, but looks pretty simple, right? :)

To post a comment you must log in.
Revision history for this message
Keegan Carruthers-Smith (keegan-csmith) :
review: Approve
Revision history for this message
Ibid Branch Auto-Lander (ibid-tarmac) wrote :

Voting does not meet specified criteria. Required: Approve >= 1, Disapprove == 0. Got: 1 Pending.

Revision history for this message
Max Rabkin (max-rabkin) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'ibid/plugins/codecontest.py'
--- ibid/plugins/codecontest.py 2010-03-27 15:50:58 +0000
+++ ibid/plugins/codecontest.py 2011-10-30 20:43:29 +0000
@@ -193,7 +193,7 @@
193 for type in ['raw', 'deaddressed', 'clean', 'stripped']:193 for type in ['raw', 'deaddressed', 'clean', 'stripped']:
194 event['message'][type] = re.sub(r'(.*)(%s)' % re.escape(term), r'\1[redacted]', event['message'][type])194 event['message'][type] = re.sub(r'(.*)(%s)' % re.escape(term), r'\1[redacted]', event['message'][type])
195195
196 @match(r'^(\S+)\s+(?:is|am)\s+(\S+)\s+on\s+usaco(?:\s+password\s+(\S+))?$')196 @match(r'^(\S+)\s+(?:is|am)\s+(\S+)\s+on\s+usaco(?:\s+password\s+(.+))?$')
197 def usaco_account(self, event, user, usaco_user, password):197 def usaco_account(self, event, user, usaco_user, password):
198 if password:198 if password:
199 self._redact(event, password)199 self._redact(event, password)

Subscribers

People subscribed via source and target branches

to all changes: