setCommentVisibility() gives 401: unauthorized error
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Medium
|
Bryce Harrington |
Bug Description
I notice the new setCommentVisib
./test > /tmp/hide_
Traceback (most recent call last):
File "./test", line 20, in <module>
d = lp.load("https:/
File "/usr/lib/
document = self._browser.
File "/usr/lib/
response, content = self._request(url, extra_headers=
File "/usr/lib/
raise HTTPError(response, content)
launchpadlib.
test script and output attached
Related branches
- Eleanor Berger (community): Approve (code)
-
Diff: 231 lines (+30/-22)9 files modifiedlib/lp/bugs/browser/bug.py (+1/-1)
lib/lp/bugs/doc/bugmessage.txt (+1/-1)
lib/lp/bugs/interfaces/bug.py (+20/-12)
lib/lp/bugs/interfaces/bugtask.py (+1/-1)
lib/lp/bugs/interfaces/malone.py (+1/-1)
lib/lp/code/doc/branch-visibility.txt (+1/-1)
lib/lp/code/model/tests/test_branchnamespace.py (+3/-3)
lib/lp/code/stories/branches/xx-branch-edit-privacy.txt (+1/-1)
lib/lp/registry/browser/team.py (+1/-1)
Changed in malone: | |
assignee: | nobody → Bryce Harrington (bryceharrington) |
Changed in malone: | |
milestone: | none → 10.05 |
Changed in malone: | |
status: | Triaged → In Progress |
Changed in malone: | |
milestone: | 10.05 → 10.06 |
Changed in malone: | |
status: | Fix Committed → Fix Released |
status: | Fix Released → Fix Committed |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in malone: | |
status: | Fix Committed → Fix Released |
setCommentVisib ility() is restricted to Launchpad admins, so this is
expected behaviour.
However, this should be mentioned in +apidoc, or another way should be
found to expose the security restrictions on API calls (like a more
informative error message).