[sponsor-patch] Traceback when user can't unsubscribe ubuntu-sponsors

Bug #896884 reported by Andreas Moog
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Fix Released
Medium
Andreas Moog

Bug Description

andreas@anubis:~/temp$ sponsor-patch 896849 -w . -u ubuntu
gpgv: Signature made Sun 06 Nov 2011 09:03:03 AM CET using RSA key ID B6E57DE1
gpgv: Can't check signature: public key not found
Use of qw(...) as parentheses is deprecated at /usr/share/lintian/collection/index line 192.

Please check transgui 3.2-1 carefully:
file:///home/andreas/temp/transgui_3.2-1.lintian
transgui has 1 opportunities: oneiric (1)
Find out more: http://harvest.ubuntu.com/opportunities/package/transgui
Do you want to acknowledge the sync request [y|e|N|?]? y
Traceback (most recent call last):
  File "/usr/bin/sponsor-patch", line 129, in <module>
    main()
  File "/usr/bin/sponsor-patch", line 120, in main
    options.upload, workdir)
  File "/usr/lib/python2.7/dist-packages/ubuntutools/sponsor_patch/sponsor_patch.py", line 276, in sponsor_patch
    if source_package.ack_sync(upload, task.get_lp_task(), launchpad):
  File "/usr/lib/python2.7/dist-packages/ubuntutools/sponsor_patch/source_package.py", line 102, in ack_sync
    bug.unsubscribe(person=launchpad.people['ubuntu-sponsors'])
  File "/usr/lib/python2.7/dist-packages/lazr/restfulclient/resource.py", line 573, in __call__
    url, in_representation, http_method, extra_headers=extra_headers)
  File "/usr/lib/python2.7/dist-packages/lazr/restfulclient/_browser.py", line 281, in _request
    raise error
lazr.restfulclient.errors.Unauthorized: HTTP Error 401: Unauthorized
Response headers:
---
-content-encoding: gzip
content-length: 74
content-type: text/plain
date: Sun, 27 Nov 2011 14:24:09 GMT
server: zope.server.http (HTTP)
status: 401
vary: Accept,Accept-Encoding
via: 1.1 api.launchpad.net
x-lazr-notifications: []
x-powered-by: Zope (www.zope.org), Python (www.python.org)
---
Response body:
---
Andreas Moog does not have permission to unsubscribe Ubuntu Sponsors Team.
---

Attached branch adds a check if the user can unsubscribe the sponsors team before attempting to do so.

Changed in ubuntu-dev-tools (Ubuntu):
importance: Undecided → Medium
status: New → Triaged
Changed in ubuntu-dev-tools (Ubuntu):
status: Triaged → Fix Committed
assignee: nobody → Andreas Moog (amoog)
Revision history for this message
Launchpad Janitor (janitor) wrote :

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-upload-permission: Query upload permissions.
    - 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

Changed in ubuntu-dev-tools (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.