[requestsync] Traceback instead of retry for 4xx response
Bug #850360 reported by
Scott Kitterman
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| ubuntu-dev-tools (Ubuntu) |
Fix Released
|
Medium
|
Andreas Moog | ||
Bug Description
When delivering by mail, if the message is deferred, requestsync should wait and retry the message rather than crash:
Press [Enter] to continue or [Ctrl-C] to abort.
Connecting to mx.launchpad.net:25 ...
Traceback (most recent call last):
File "/usr/bin/
main()
File "/usr/bin/
mailserver_
File "/usr/lib/
s = smtplib.
File "/usr/lib/
raise SMTPConnectErro
smtplib.
Related branches
lp://staging/~ampelbein/ubuntu-dev-tools/fix-850360
- Stefano Rivera: Approve
-
Diff: 86 lines (+43/-10)2 files modifieddebian/changelog (+1/-0)
ubuntutools/requestsync/mail.py (+42/-10)
| Changed in ubuntu-dev-tools (Ubuntu): | |
| importance: | Undecided → Medium |
| Changed in ubuntu-dev-tools (Ubuntu): | |
| status: | New → Fix Committed |
| assignee: | nobody → Andreas Moog (amoog) |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #1 |
| Changed in ubuntu-dev-tools (Ubuntu): | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This bug was fixed in the package ubuntu-dev-tools - 0.137
---------------
ubuntu-dev-tools (0.137) unstable; urgency=low
[ Stefano Rivera ]
* mk-sbuild: Make --eatmydata command line flag actually work.
* Remove dgetlp. No longer needed.
* Use httplib2 everywhere that we do https. The python stdlib doesn't do
certificate verification.
* requestbackport:
- Check for existing backport bugs first.
- Check that there are published binaries (i.e. not stuck in bin-NEW).
* pull-lp-source, requestbackport: Take the latest version from any
non-backports pocket. Implemented by making lpapicache's getSourcePackage
smarter.
* sponsor-patch: Build a source package for lintian to run on, when not
test-building syncs.
* sponsor-patch: Check the bug's title, not the task, when determining
source series for syncs.
* mk-sbuild, pbuilder-dist, ubuntu-build: Add armhf.
* pull-debian-source, pull-lp-source: Resolve the source package (via DDE),
if a binary package was requested (LP: #617349)
* submittodebian:
- Do the report boiler-plate checking in a script that wraps an editor, so
that we only edit the report once, after checking for duplicates.
- rm the tmpdir with a little more force (shutil.rmtree) (LP: #899399)
* New Tools: (LP: #876554)
- ubuntu-
- seeded-in-ubuntu: Query a package's seed status. Whether it is on
current daily images and/or part of the supported seed.
* syncpackage: Support sponsorship for native-syncs, now that LP does.
[ Andreas Moog ]
* sponsor-patch: Check permission to unsubscribe sponsors-team (LP: #896884)
* grep-merges: We already require a UTF-8 enabled terminal, so encode
package and uploader name in UTF-8 (LP: #694388)
* requestsync: Give user option to retry in case of temporary error
(LP: #850360)
-- Stefano Rivera <email address hidden> Fri, 09 Dec 2011 12:59:29 +0200