Current Drupal OpenID implementation supports only SREG for requesting/receiving credential from OpenID provider. This patch adds AX support as well. In practice this means that now Drupal OpenID authentication requests email from Google, when it is used as OpenID provider.
This patch is for Drupal 6.x. I'll prepare patch with the same feature for D7 (if not implemented yet) little later.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | openid-ax-support-4-d6.patch | 5.62 KB | vgarvardt |
| #4 | openid-ax-support-3-d6.patch | 5.29 KB | vgarvardt |
| #1 | openid-ax-support-d6-2.patch | 5.76 KB | vgarvardt |
| openid-ax-support.patch | 5.77 KB | vgarvardt |
Comments
Comment #1
vgarvardt commentedSorry, invalid patch from Mercurial diff. Here is updated patch
Comment #2
damien tournoud commentedAwesome, thanks for the patch, but this has to be considered first for D7, before being eventually backported.
Comment #3
vgarvardt commentedI already created same patch for D7 OpenID version - #634562: OpenID AX support
Comment #4
vgarvardt commentedHere is D6 patch backport from D7 patch that passed all tests.
Comment #5
vgarvardt commentedUpdated patch that applies to D 6.15
Comment #6
RikaWatanabe commentedThat no longer works with Google in particular because they changed their discovery protocol slightly: #727650: Support Google-specific OpenID discovery protocol
Is there a chance for an update to this patch?
Comment #7
damien tournoud commentedThis is a duplicate of #634562: OpenID AX support.
Comment #8
tuffnatty commented#6: It works for me.
Comment #9
petey318 commentedI know this has been marked as a duplicate, but will this patch be rolled into D6? we are currently at d6.17 but still the core openid module does not appear to support AX with google.
The duplicate ( #634562: OpenID AX support ) actually refers to D7 btw...
Comment #10
vgarvardt commentedIf you are interested in the AX support for D6 - see #634562: OpenID AX support, latest backported patches will be posted there.