requestsync fails with 401 unauthorized when adding subscribers

[Iain Lane]

Binary package hint: ubuntu-dev-tools

Hi,

When filing a sync, requestsync dies when trying to set the importance if the submitter doesn't have permission (I don't). Traceback:

Traceback (most recent call last):
  File "/home/laney/bin/ubuntu-dev-tools/requestsync", line 562, in <module>
    if post_bug(srcpkg, subscribe, mapping[status], title, report):
  File "/home/laney/bin/ubuntu-dev-tools/requestsync", line 388, in post_bug
    task.transitionToImportance(importance='Wishlist')
  File "/usr/lib/python2.5/site-packages/launchpadlib/resource.py", line 336, in __call__
    url, in_representation, http_method, extra_headers=extra_headers)
  File "/usr/lib/python2.5/site-packages/launchpadlib/_browser.py", line 211, in _request
    raise HTTPError(response, content)
launchpadlib.errors.HTTPError: HTTP Error 401: Unauthorized

The bug is created. I guess this exception should be trapped and ignored.

Running bzr trunk.

laney@chicken:~/dev/ubuntu/packaging/darcs$ apt-cache policy python-launchpadlib
python-launchpadlib:
  Installed: 0.2~bzr25-0ubuntu1
  Candidate: 0.2~bzr25-0ubuntu1
  Version table:
 *** 0.2~bzr25-0ubuntu1 0
        500 http://archive.ubuntu.com intrepid/universe Packages
        100 /var/lib/dpkg/status

[Jonathan Davies (jpds)]

Merged into main branch, thanks.

------------------------------------------------------------
revno: 305
committer: Jonathan Davies <email address hidden>
branch nick: ubuntu-dev-tools
timestamp: Sun 2009-01-25 10:11:06 +0000
message:
  Merge in Iain's changes.
    ------------------------------------------------------------
    revno: 263.1.7
    committer: Iain Lane <email address hidden>
    branch nick: ubuntu-dev-tools
    timestamp: Sun 2009-01-25 01:07:51 +0000
    message:
        will fail otherwise (LP: #320984).
    ------------------------------------------------------------
    revno: 263.1.6
    committer: Iain Lane <email address hidden>
    branch nick: ubuntu-dev-tools
    timestamp: Sun 2009-01-25 01:05:02 +0000
    message:
      ubuntutools/lp/functions.py: Rename urlopener import as it conflicts with
      a variable, causing an error.
    ------------------------------------------------------------
    revno: 263.1.5
    committer: Iain Lane <email address hidden>
    branch nick: ubuntu-dev-tools
    timestamp: Sun 2009-01-25 00:15:06 +0000
    message:
      requestsync: Only attempt to change bug importance if in ubuntu-dev, as it
      will fail otherwise.
    ------------------------------------------------------------

[Launchpad Janitor (janitor)]

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

---------------
ubuntu-dev-tools (0.60) jaunty; urgency=low

  [ Jonathan Davies ]
  * ubuntutools/common.py: Now split into multiple files depending on
    function.
  * Adjusted imports on all files as necessary for the change above.
  * Removed ubuntutools/misc.py's mkdir function - superseded by
    os.makedirs().
  * dgetlp: Improved error message to show that dgetlp only accepts HTTP
    URLs (LP: #322051).

  [ Iain Lane ]
  * requestsync: Only attempt to change bug importance if in ubuntu-dev, as it
    will fail otherwise (LP: #320984).
  * ubuntutools/lp/functions.py: Rename urlopener import as it conflicts with
    a variable, causing an error.

  [ Luca Falavigna ]
  * pull-debian-source: do not fail if package name contains a hypen.
  * buildd: display help message if no parameters are passed.

 -- Jonathan Davies <email address hidden> Sun, 01 Feb 2009 10:55:42 +0000