Support XRDS Yadis response to be compatible with module openid

schuyler1d - October 26, 2008 - 21:56
Project:OpenID URL
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:critical
Assigned:Unassigned
Status:closed
Issue tags:OpenID, XRDS Simple
Description

The OpenID drupal module (version 1.2 for 5.x and maybe for 6.x) seems to require an xrds document.
This supports HTTP requests with 'Accept: application/xrds+xml' which should also improve performance, since this is often attempted before looking through html headers.

The patch adds a default-off (for backwards compatibility) feature to enable XRDS response from the home page.

AttachmentSize
xrds_yadis.patch2.22 KB

#1

Wim Leers - March 11, 2009 - 23:08
Title:support XRDS Yadis response to be compatible with module openid» Support XRDS Yadis response to be compatible with module openid
Status:needs review» needs work

Is there a reason that this needs to be a setting? Can't we just enable this *always*?

#2

Rob Loach - October 4, 2009 - 09:01
Version:5.x-1.x-dev» 6.x-1.x-dev
Priority:normal» critical
Status:needs work» needs review

Seems like we'd want to move over to the XRDS Simple module. Pushing to critical before next release.

AttachmentSize
openidurlxrds.patch 6.93 KB

#3

Rob Loach - October 5, 2009 - 00:34
Status:needs review» fixed

Committed to the DRUPAL-6--2 branch.

#4

System Message - October 19, 2009 - 00:40
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.