[development] OpenID

[Ainsley Pereira]

On Fri, Feb 10, 2006 at 01:01:12PM -0800, Jonathan Daugherty wrote:
> # I couldn't find Auth_OpenID_DatabaseConnection anywhere, which
> # Drupal_OpenID_DBConnection attempts to extend
> 
> Well, I didn't expect anyone to actually try running the module; it's
> just for viewing and has been developed against an unreleased version
> of our PHP OpenID library. :) I guess what I am looking for (with
> regard to the code specifically) is "that should work and is
> acceptable", "don't do that, that's terribly awful", or "You'e almost
> there, but here are some things to consider."  But I'll take a test
> run, too, for sure. :)

Ah, well, I don't know a lot about Drupal internals personally, but as
I'm interested in getting OpenID going with it, I thought I could best
help by trying it out and offering encouragement :)
 
> Thanks for the detailed reply, and thanks for going to the trouble to
> get it to work.  Your decision to extend DB_mysql was correct.  Check
> to be sure that the openid_* tables are in your Drupal database.  Did
> you get any kind of error when you return to your site from the OpenID
> server?  (If you made it that far, things appear to be in working
> order.)  When you return to your test site, do you get prompted to
> enter an email address for the new account?  If so, does that succeed?

Yes, I have three openid_consumer_* tables, settings and associations
each have one entry (settings has an auth_key, associations has an entry
with the server_url I used. No error shown, in the logs I see some
'table already exists' messages, and a 'No colon on line 6:' from
KVForm.php line 77. I did get prompted for an email address, but no
account with that address is in my users table.

After a bit more investigation it seems that it was a browser issue.
Konqueror 3.5.1 doesn't like it somehow, but with Firefox it works OK.
(Except for there being an extra 'openid/' in the returned-to url, which
I can fix by removing it from the header("Location: "... line in
openid_return() ).

I'll try looking into the Konqueror problem myself.

~Ainsley