I noticed that Drupal module OpenID doesn't recognize openid.pl domain
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | 224793.patch | 2.43 KB | robloach |
I noticed that Drupal module OpenID doesn't recognize openid.pl domain
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | 224793.patch | 2.43 KB | robloach |
Comments
Comment #1
robloachOpenID URL implements both OpenID 1 and 2 specs, so I'm not sure what's wrong. In the very least, try updating to a later version of the module.
Comment #2
robloachCould you try it again? I just put in a patch that switches the attributes around... http://drupal.org/node/229877
Comment #3
robloachSome hosts, like sourceforge, don't support having the two links listed in the same variable, so we should split them up.
Comment #4
Inte commentedI use phpMyID which only supports OpenID 1.1 (AFAIK) and run into some problems.
Removing
openid2.providerandopenid2.local_idin/sites/all/modules/openidurl/openidurl.modulesolved it for me.Maybe you can add a selection, which OpenID-HTML-Headers get included.
For example:
Comment #5
robloachGreat idea, Inte.
Comment #6
robloachI'll hit it up today. Mind testing 6.x-1.x-dev once it's in? I'd like to do a release sooner then later.
Comment #7
robloachActually, could you test this with 6.x-1.x-dev now? It already splits the parameters into two different link tags instead of sticking them in the same parameter.....
If that works for you, I'll make a release.....
Comment #8
robloach............ Dev should be good..... If not, this'll do it.
Comment #9
robloachAfter some testing on SourceForge, it still seems to be a problem, so I committed this.
http://drupal.org/cvs?commit=139652
Comment #10
robloachIf anyone wants this in the Drupal 5 branch, feel free to send in a patch.
Comment #11
Inte commentedI checked the OpenID 1.1 settings as phpMyID doesn't support OpenID 2.0
It does it's job ... nothing more and nothing less. ;) Thanks for your work.
Maybe someone with a 1.1 & 2.0 compatible provider should run a check including both parameters, before you release a stable version (which I'm waiting for).
Comment #12
robloachComment #14
ao2 commentedThe issue has been fixed for quite some time, let's update the status to reflect that.