Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

server enhancement, accepted signature methods #2

Closed
GoogleCodeExporter opened this issue Mar 21, 2015 · 4 comments
Closed

server enhancement, accepted signature methods #2

GoogleCodeExporter opened this issue Mar 21, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

A service provider should be able to set the accepted signature methods.
For example I would like to accept PLAINTEXT only for SSL encrypted requests.

proposal:
extend OAuthRequestVerifier with a method
OAuthRequestVerifier->setAcceptedSignatureMethod() taking an array of
strings (the signature methods), saving this in a class variable. if null
is passed all supported methods are accepted and there is no filtering.

OAuthRequestVerifier->verify should check this value and filter the request
signature method and throw an error (just like how it is node in
OAuthRequest for available methods).

alternatively the opposite could be done and a ->setDeniedSignatureMethod()
added and filtering upon that.

-ueli

Original issue reported on code.google.com by uwe...@gmail.com on 17 Jul 2008 at 3:03

@GoogleCodeExporter
Copy link
Author

comment to clear up the nature of this post: this is not a bug, it's just a
low-priority enhancement. Service Provider implementers can access
$r->getParam('oauth_signature_method') and filter the signature method 
themselves,
after initialization. 

Original comment by uwe...@gmail.com on 17 Jul 2008 at 3:19

@GoogleCodeExporter
Copy link
Author

Sorry to react so late :-)   I didn't forget about your enhancement.  I like 
the idea to prevent PLAINTEXT with non 
SSL connections, makes a lot of sense.  I will check with scherpenisse about 
your suggestion and a possible way 
to do this.

Original comment by ma...@pobox.com on 25 Jan 2009 at 11:24

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

Fixed in SVN.

Original comment by brunobg%...@gtempaccount.com on 12 Jan 2010 at 7:19

  • Changed state: Fixed
  • Added labels: Type-Enhancement
  • Removed labels: Type-Defect

@GoogleCodeExporter
Copy link
Author

This issue was closed by revision r69.

Original comment by brunobg%...@gtempaccount.com on 12 Jan 2010 at 7:19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant