isitdeployable

Merge lp:~roadmr/isitdeployable/badrequest into lp:isitdeployable

  1. badrequest
  2. Merge into trunk
Proposed by Daniel Manrique
Status: Merged
Approved by: Daniel Manrique
Approved revision: 301
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: lp:~roadmr/isitdeployable/badrequest
Merge into: lp:isitdeployable
Diff against target: 16 lines (+5/-1)
1 file modified
revtracker/tasks.py (+5/-1)
To merge this branch: bzr merge lp:~roadmr/isitdeployable/badrequest
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Review via email: mp+382710@code.launchpad.net

Commit message

Add some temporary debugging to find email address which is causing a BadRequest from Launchpad.

To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

Self-approve trivial and I'll probably even revert once I pinpoint the actual problem and can write something better.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'revtracker/tasks.py'
2--- revtracker/tasks.py 2019-09-06 22:15:47 +0000
3+++ revtracker/tasks.py 2020-04-21 21:52:15 +0000
4@@ -449,7 +449,11 @@
5 if "." not in email:
6 assignee = None
7 else:
8- assignee = lp.people.getByEmail(email=email)
9+ try:
10+ assignee = lp.people.getByEmail(email=email)
11+ except Exception as err:
12+ print("Error %r for %s" % (err, email))
13+
14 email_map[email] = assignee
15 if assignee is None:
16 return None

Subscribers

People subscribed via source and target branches