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.

CommentFileSizeAuthor
#5 openid-ax-support-4-d6.patch5.62 KBvgarvardt
#4 openid-ax-support-3-d6.patch5.29 KBvgarvardt
#1 openid-ax-support-d6-2.patch5.76 KBvgarvardt
openid-ax-support.patch5.77 KBvgarvardt

Comments

vgarvardt’s picture

vgarvardt commented
StatusFileSize
new openid-ax-support-d6-2.patch5.76 KB

Sorry, invalid patch from Mercurial diff. Here is updated patch

damien tournoud’s picture

damien tournoud commented
Version: 6.x-dev » 7.x-dev
Status: Needs review » Patch (to be ported)

Awesome, thanks for the patch, but this has to be considered first for D7, before being eventually backported.

vgarvardt’s picture

vgarvardt commented

I already created same patch for D7 OpenID version - #634562: OpenID AX support

vgarvardt’s picture

vgarvardt commented
Version: 7.x-dev » 6.x-dev
Status: Patch (to be ported) » Needs review
StatusFileSize
new openid-ax-support-3-d6.patch5.29 KB

Here is D6 patch backport from D7 patch that passed all tests.

vgarvardt’s picture

vgarvardt commented
StatusFileSize
new openid-ax-support-4-d6.patch5.62 KB

Updated patch that applies to D 6.15

RikaWatanabe’s picture

RikaWatanabe commented

That 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?

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Closed (duplicate)

This is a duplicate of #634562: OpenID AX support.

tuffnatty’s picture

tuffnatty commented

#6: It works for me.

petey318’s picture

petey318 commented

I 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...

vgarvardt’s picture

vgarvardt commented

If you are interested in the AX support for D6 - see #634562: OpenID AX support, latest backported patches will be posted there.