Modify

Opened 16 years ago

Closed 16 years ago

#3706 closed enhancement (wontfix)

AuthOpenID uses strange username

Reported by: dsvensson@… Owned by: Dalius
Priority: low Component: AuthOpenIdPlugin
Severity: minor Keywords:
Cc: Trac Release: 0.11

Description

I use AuthOpenID with wordpress and my username becomes http://dsvensson.wordpress.com/ which looks strange. There are config options to remove the protocol and the trailing slash and then the username becomes dsvensson.wordpress.com. These config settings are a hack and the plugin should instead request the username from the openid site it communicates with. In my case it should have grabbed both my IRL name, and the wordpress username, and associated my IRL name with my wordpress username and used the username as trac username. So simply dsvensson.

Attachments (2)

out.gif (19.4 KB) - added by anonymous 16 years ago.
wordpress import screen, including my short username (oid for stackoverflow.com)
out2.gif (18.1 KB) - added by anonymous 16 years ago.
what ends up in my profile at stackoverflow.com after the wordpress import, nice username, but also image (there's another trac hack for using profile images right?)

Download all attachments as: .zip

Change History (7)

comment:1 Changed 16 years ago by Dalius

Resolution: invalid
Status: newclosed

This plugin requests name from site and uses it as Full Name!!!

I'm not sure if you are not confusing Full Name (from preferences) with username. username MUST BE openid (with or without http) because otherwise dsvensson.myopenid.com (dsvensson.anything.* in general) could log-in into trac site as you. Full name from other side is used by trac (Logged in as Dalius) but for example near tickets if shows openid (owner for example shows username instead of Full Name ). That's trac's problem not this plugin. I will not do stupid things with this plugin - fork it if you want to do unsecure stuff.

Try login using openid here and see what I mean: http://trac.sandbox.lt/skaityta/

comment:2 Changed 16 years ago by anonymous

I'm logged in as dsvensson.wordpress.com in my trac, and my full name is empty even though my full name is entered into wordpress.com account. Also, when I'm using other OpenID sites, such as stackoverflow.com, it uses my wordpress account name as account name there, that is.. simply "dsvensson". This is the functionality I'm looking for in this plugin too. Can't "http://dsvensson.wordpress.com/" just be a login handle, and then the real account would use dsvensson + Daniel Svensson, and if the tuple ("dsvensson","http://dsvensson.wordpress.com/") already exists on the Trac, then the user would be presented with a request to enter another name, such as perhaps ("dsv", "http://dsvensson.someothersite.com").

comment:3 Changed 16 years ago by Dalius

Priority: normallow
Resolution: invalid
Severity: normalminor
Status: closedreopened
Type: defectenhancement

Most probably that's possible with some hacking but I don't see it worth my time. If you have enough time you can program it - it is open source after all :-) You will need to create mappings table (map between openid and username) and handle conflict situations. It might be completely possible that it is not very complicated.

Changed 16 years ago by anonymous

Attachment: out.gif added

wordpress import screen, including my short username (oid for stackoverflow.com)

Changed 16 years ago by anonymous

Attachment: out2.gif added

what ends up in my profile at stackoverflow.com after the wordpress import, nice username, but also image (there's another trac hack for using profile images right?)

comment:4 Changed 16 years ago by anonymous

Just added the two above images to demonstrate what's possible. Using OpenID with bug trackers is extremly important as it makes it a lot easier to contribute to open source projects. Thanks a lot for your effort. I really hope this plugin ends up in Trac by default.

comment:5 Changed 16 years ago by Dalius

Resolution: wontfix
Status: reopenedclosed

I was thinking about this ticket for some time and I will not fix it.

It is more reasonable to register such ticket to Trac itself. Should sound something like that:

--- Show my "Full name" instead of my username everywhere. "Full name" should be link that points to user information page. There is no such page most probably I guess. Such page could contain information about owned and/or created tickets. ---

Svensson, if you have time please search through trac tickets http://trac.edgewall.org/report/1 and look if they don't have something similar already.

I don't want to make trac plugin heavy weight. Even if it is possible with conflict resolution I find it ugly as solution. Reasons:

1) conflict resolution will put user one step further from filling ticket;

2) OpenID nickname attribute always can be changed. We have problem here handling nickname change and again new conflict resolution.

That will make openid plugin unnecessary complicated and will increase its maintenance price. While price for users is zero it is not zero for me - I do it on my freetime. In addition it will not push trac team to make right decissions ;-) The last time I have looked into trac code I found its design a little bit wrong. "Full name" in trac is session attribute but not user attribute if I remember it correctly. That means that with session expiration "Full name" might be lost.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Dalius.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.