Ubuntu
ubuntu-dev-tools package

requestsync --lp -s aborts on setting importance when submitter lacks permission

Bug #374563 reported by Anders Kaseorg
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: ubuntu-dev-tools

requestsync in ubuntu-dev-tools 0.73 does not deal well when the submitter does not have permission to set the importance of the bug they just opened. (This is possibly a reprise of bug #190061, but I’m opening a new bug because that one is old.)

$ requestsync --lp -s git-core karmic
…
Traceback (most recent call last):
  File "/usr/bin/requestsync", line 612, in <module>
    if post_bug(srcpkg, subscribe, mapping[status], title, report):
  File "/usr/bin/requestsync", line 407, in post_bug
    task.transitionToImportance(importance='Wishlist')
  File "/usr/lib/python2.6/dist-packages/launchpadlib/resource.py", line 349, in __call__
    url, in_representation, http_method, extra_headers=extra_headers)
  File "/usr/lib/python2.6/dist-packages/launchpadlib/_browser.py", line 212, in _request
    raise HTTPError(response, content)
launchpadlib.errors.HTTPError: HTTP Error 401: Unauthorized

Because it aborted early, it failed to subscribe ubuntu-main-sponsors to bug #374558, and I had to do so manually.

Revision history for this message
Iain Lane (laney) wrote :

Unmarking as dupe - those are actually different bugs

Changed in ubuntu-dev-tools (Ubuntu):
importance: Undecided → Medium
status: New → Fix Committed
Revision history for this message
Iain Lane (laney) wrote :

Fixed in r394, which was committed yesterday. Timely bug reporting.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-dev-tools - 0.74

---------------
ubuntu-dev-tools (0.74) karmic; urgency=low

  [ Kees Cook ]
  * mk-sbuild-lv:
    - Skip security repo for Debian unstable, thanks to Ryan Niebur
      (LP: #371569).
    - Change directory out of the way of schroot problems.

  [ Siegfried-Angel Gevatter Pujals ]
  * grab-merge:
     - Show an error message if the package doesn't exist.
     - Be paraonic and add "--one-file-system" to the rm call.
     - Delete the directory just after creating it if the package
       doesn't exist.

  [ Iain Lane ]
  * ubuntutools/lp/lp_functions.py,
    ubuntutools/lp/udtexceptions.py:
    - Add new public functions that expose features from LP API
    - Modify isLPTeamMember to use LP API
  * requestsync
    - Use new functions to check if user can upload requested package directly
      instead of checking team membership
    - Default to current development release if no release is specified on
      commandline
    - Correct bug supervisor team to ubuntu-bugcontrol (LP: #374563)
    - Remove team names from sponsorship message - makes the function much
      simpler
  * buildd
    - Check if user has upload privileges instead of checking for team
      membership when seeing if operations are permitted

  [ Colin Watson ]
  * update-maintainer:
    - Convert to getopt so that '--section main' works as well as
      '--section=main'.

  [ Anders Kaseorg ]
  * ubuntutools/lp/functions.py:
    - Simplify isLPTeamMember.

  [ Nathan Handler ]
  * pull-debian-source: Modify to work for packages not in main (LP: #379822)

 -- Nathan Handler <email address hidden> Sat, 23 May 2009 20:41:50 +0000

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.
Subscribing...

Other bug subscribers

Remote bug watches

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