+class OAuthTokenBase(OAuthBase):
+ """Base implementation of code to check an OAuth-signed request."""
I'd change that docstring to say just that this is a base class for OAuthToken classes, as we may want to add code not related to signature checking here in the future.
And here you should add a docstring pointing to the interface where the method is defined.
-Now consider a principal authorized to create OAuth tokens. Whenever
-it's not creating OAuth tokens, it has a level of permission
-equivalent to READ_PUBLIC.
+A principal with the GRANT_PERMISSIONS authorization level has a of
+permission equivalent to WRITE_PRIVATE.
Why is the permission changing in this incremental diff? Also, s/of//?
And here's another follow up review: http:// pastebin. ubuntu. com/486932/
+class OAuthTokenBase( OAuthBase) :
+ """Base implementation of code to check an OAuth-signed request."""
I'd change that docstring to say just that this is a base class for OAuthToken classes, as we may want to add code not related to signature checking here in the future.
+ self, request): signature( request, self.consumer, self.secret)
+ def checkSignature(
+ return check_oauth_
And here you should add a docstring pointing to the interface where the method is defined.
-Now consider a principal authorized to create OAuth tokens. Whenever
-it's not creating OAuth tokens, it has a level of permission
-equivalent to READ_PUBLIC.
+A principal with the GRANT_PERMISSIONS authorization level has a of
+permission equivalent to WRITE_PRIVATE.
Why is the permission changing in this incremental diff? Also, s/of//?
- >>> access_token = token.createAcc essToken( ) roxy(token. createAccessTok en())
+ >>> access_token = removeSecurityP
Why do you need to remove the security proxy now?