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.
Current version is unable to authenticate with 1.1 servers (draft Blogger as an example), because it sends unknown fields for 1.1 specification. http://openid.net/specs/openid-authentication-1_1.html#mode_checkid_imme... The patch fixes that.
Comment | File | Size | Author |
---|---|---|---|
openid.module.patch.txt | 1.32 KB | techtonik |
Comments
Comment #1
techtonik CreditAttribution: techtonik commentedCould somebody with write access add this to some release branch finally?
The specific error displayed by Blogger for unpatched Drupal that sends invalid request to 1.1 server is:
error:Invalid AuthRequest: 768: Invalid value for openid.ns field: http://openid.net/signon/1.0
Comment #2
techtonik CreditAttribution: techtonik commentedComment #3
boxfire CreditAttribution: boxfire commentedCould someone possibly have a patch for version 6 please? People who want to move their blogs from blogger and considering drupal would find this very useful. Thanks
Comment #4
haxney CreditAttribution: haxney commentedApplying this fix to 6.6 worked for me, though I had to enter my OpenID twice, with the login succeeding on the second try. Annoying, but livable.
Comment #5
Vacilando CreditAttribution: Vacilando commentedSame problem in Drupal 6 :-(
chrono325, could you please post your function openid_authentication_request from openid.module. When I tried to apply the D5 patch in my D6.6, it still did not work for me. But D6 openid.module is a bit different, so I would like to see the code that is working for you.
Comment #6
Russell McOrmond CreditAttribution: Russell McOrmond commentedI would like to confirm that this problem still existed with openid-5.x-1.2 , and that the patch seemed to allow me to add http://mcormond.blogspot.com/ as an OpenID.
Comment #7
Leeteq CreditAttribution: Leeteq commentedSubscribing.
Comment #8
anarcat CreditAttribution: anarcat commentedSubscribing.
For D6, it's probably better to resubmit the patch in the Drupal core project itself, and have it ported to D7. Only then it *may* be backported to D6.
Comment #9
paganwinter CreditAttribution: paganwinter commentedI have used the above patch.
I get this after getting authenticated from Flickr:
The Terms of Condition is coming from the Legal module.
And mail id is a mandatory field for Drupal.
But when i proceeded with the Drupal registration form and register myself, I am shown the OpenId I had provided.
So would it be ok if I just suppress the above message and tell the user that the authentication was successful?
If it's only the above stated 2 reasons, then the user should be able to procedd and fill in the Drupal registration form.
But what if there was some other error that occurred while authentication?
Is there some way of finding that out and only then displaying the Failure notification?
Comment #10
paulconolly CreditAttribution: paulconolly commentedI used the patch in v5 and v6. The patch for 6 is slightly different from 5. (see the diff log below)
http://www.heronsoftware.co.uk/trac/projects/logs/drupal6_openid_link1.1...
I made a copy of the core module and worked on that, hence the path in the log is not /modules/openid.
I've tested both versions and they seem to be working fine.
Comment #11
Heine CreditAttribution: Heine commentedDuplicate of #222044: Openid login failed with Blogger users
Comment #12
Heine CreditAttribution: Heine commentedEh, not a duplicate. Best to repro patch from #222044: Openid login failed with Blogger users though.