Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
There is no definition of OAuthSignatureMethod
which means if I did new OAuthSignatureMethod_HMAC('SHA1');
I get an error because that class extends OAuthSignatureMethod
, which is not defined in the autoloader hook.
I've not included this as a patch as its a 1 line fix...
Comment | File | Size | Author |
---|---|---|---|
#2 | 899506.patch | 844 bytes | ezra-g |
#1 | 899506.patch | 717 bytes | ezra-g |
Comments
Comment #1
ezra-g CreditAttribution: ezra-g commentedHere's a patch that implements this change.
I ran into this working on #404470: Authenticate Twitter_Actions.module via OAuth.module 2.x instead of Basic Auth.
Comment #2
ezra-g CreditAttribution: ezra-g commentedHere's a more correct patch that implements this change as part of oauth_common_autoload_info().
Comment #3
gregglesThe fact that this is needed for #404470: Authenticate Twitter_Actions.module via OAuth.module 2.x instead of Basic Auth means that lots of folks are going to want this fixed in short order since there are 15,000 users of the twitter.module who need it.
Also, subscribing ;)
Comment #4
voxpelli CreditAttribution: voxpelli commentedAll implementations yet has already had that class loaded when they needed to use it due to it being in the same file as a lot of other OAuth classes, so might be that something is a bit odd with the current implementation that #404470: Authenticate Twitter_Actions.module via OAuth.module 2.x instead of Basic Auth is working on.
I'll try to keep track of that issue and I will fix this issue the next time I do commits for the OAuth module.
Comment #5
voxpelli CreditAttribution: voxpelli commentedTo clarify: This is not needed for #404470: Authenticate Twitter_Actions.module via OAuth.module 2.x instead of Basic Auth as that issue is about fixing the current support for OAuth 2.x and not to implement new support for OAuth 3.x.
Comment #6
ArtusamakLittle up on the subject, is it planned to add this patch soon? It's definitely needed. ;)
Thanks.
Comment #7
voxpelli CreditAttribution: voxpelli commentedIt will be in the next release
Comment #8
voxpelli CreditAttribution: voxpelli commentedThis has been commited and will appear very soon in 6.x-3.0-beta3 - thanks for the patch and help!