Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.4.0

Bug #19234 Normalization of identifiers/openID not implemented correctly
Submitted: 2012-01-22 04:21 UTC
From: cweiske Assigned: cweiske
Status: Closed Package: OpenID (version 0.3.2)
PHP Version: Irrelevant OS:
Roadmaps: 0.4.0    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 48 - 31 = ?

 
 [2012-01-22 04:21 UTC] cweiske (Christian Weiske)
Description: ------------ I'm running http://shupp.org/openid/examples/relyingparty.php with "cweiske.de" as my identifier (OpenID). According to the specs[1], this should be normalized to http://cweiske.de/ (note the trailing slash). The OpenID package shows in debug mode: > openid.claimed_id http://cweiske.de > openid.identity http://cweiske.de (slash missing) which leads to my OpenID server telling me > Your current identity does not match the requested identity http://cweiske.de. [1] http://openid.net/specs/openid-authentication-2_0.html#normalization_example

Comments

 [2012-01-22 04:40 UTC] cweiske (Christian Weiske)
I fixed it in a separate branch: https://github.com/pear/openid/tree/bug19234 Bill, please review and merge it.
 [2012-10-14 22:26 UTC] till (Till Klampaeckel)
Christian, is this merged already? If so, can you close this? Till
 [2012-10-14 22:27 UTC] till (Till Klampaeckel)
-Status: Open +Status: Feedback -Assigned To: +Assigned To: cweiske
 [2012-10-29 23:52 UTC] cweiske (Christian Weiske)
-Status: Feedback +Status: Open
yes, it's merged already.
 [2012-10-29 23:52 UTC] cweiske (Christian Weiske)
-Status: Assigned +Status: Closed