lib/canonical/launchpad/pagetests/webservice/oauth.txt and most of the changes to lib/canonical/launchpad/doc/oauth.txt look like their primary purpose is to provide test coverage, not to provide documentation, so please write them as UnitTests.
It appears that the removeSecurityProxy in lib/canonical/launchpad/webapp/servers.py could be eliminated by moving some of the code into a method on OAuthAccessToken. Please try this, and if it's not possible, provide a follow-up comment explaining this.
lib/canonical/ launchpad/ pagetests/ webservice/ oauth.txt and most of the changes to lib/canonical/ launchpad/ doc/oauth. txt look like their primary purpose is to provide test coverage, not to provide documentation, so please write them as UnitTests.
A lot of the formatting of line-wraps doesn't match our official style: https:/ /dev.launchpad. net/PythonStyle Guide#Whitespac e%20and% 20Wrapping and often matches the "Do not indent your code like this" style. Please fix.
It appears that the removeSecurityProxy in lib/canonical/ launchpad/ webapp/ servers. py could be eliminated by moving some of the code into a method on OAuthAccessToken. Please try this, and if it's not possible, provide a follow-up comment explaining this.