Merge lp:~bac/launchpad/bug-750984 into lp:launchpad
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Brad Crittenden | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 13127 | ||||
Proposed branch: | lp:~bac/launchpad/bug-750984 | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
418 lines (+92/-55) 3 files modified
lib/canonical/launchpad/doc/webapp-publication.txt (+50/-33) lib/canonical/launchpad/interfaces/oauth.py (+22/-6) lib/canonical/launchpad/webapp/servers.py (+20/-16) |
||||
To merge this branch: | bzr merge lp:~bac/launchpad/bug-750984 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
j.c.sackett (community) | Approve | ||
Review via email: mp+62543@code.launchpad.net |
Commit message
[r=jcsackett][bug=750984] Don't OOPS on bad API token/nonce/
Description of the change
= Summary =
Previously problems with API token, nonce, or timestamp resulted in an
Unauthorized exception being raised which caused an OOPS to be
generated. Since the likely cause of those problems is bad client data
(though server side processing could be at fault) it is inappropriate to
generated an OOPS.
== Proposed fix ==
Rather than returning Unauthorized return custom exceptions that have
been marked with webservice_error as 401. The client still sees an HTTP
401 but no OOPS is reported.
== Pre-implementation notes ==
Talks with Gary and lots of conversation on the bug between Robert and
Martin.
== Tests ==
bin/test -vvt webapp-
== Demo and Q/A ==
It should be possible to force bad client data but the details are not
obvious at the moment.
= Launchpad lint =
The lint issues are not fixable.
Checking for conflicts and issues in changed files.
Linting changed files:
lib/canonical
lib/canonical
lib/canonical
./lib/canonical
195: want exceeds 78 characters.
203: want exceeds 78 characters.
325: want exceeds 78 characters.
This looks good to land.