Make OpenID identifier human readable

Bug #236194 reported by Francis J. Lacoste
4
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Curtis Hovey

Bug Description

As a user of the Launchpad Login service
I want my permanent identity url to be https://id.launchpad.net/NNN/name
so that it is meaningful in cases where RPs display it and so that it is easier to remember.

This should only affect new accounts. See bug for the migration aspect in bug #236193.

https://launchpad.canonical.com/OpenIDBetaTwo provides more background info on this change.

Changed in launchpad:
importance: Undecided → High
milestone: none → 1.2.6
status: New → Confirmed
Changed in launchpad:
assignee: nobody → sinzui-is
Revision history for this message
Stuart Bishop (stub) wrote :

This should wait until the first auth-person-split branch lands, or be worked on with that branch as a basis. It should land on June 13th 2008. The stored procedure that generates the default openid identifiers should be removed and the default value generated by Launchpad.

Are we going to allow changing the permanent identity URL? If so, we have 1 week to modify the DB design to archive old OpenID identifiers so they continue to function. This would make it unnecessary to migrate the Canonical shop database.

Revision history for this message
Curtis Hovey (sinzui) wrote :

This must be done after the we are recording OpenID identifiers.

Changed in launchpad:
milestone: 1.2.6 → none
Curtis Hovey (sinzui)
Changed in launchpad:
milestone: none → 1.99
Revision history for this message
Curtis Hovey (sinzui) wrote :

I have a branch that refactors the code to isolate the rules for working with OpenID indentity URL to a single object. Added a new URL format will be easy to implement.

Changed in launchpad:
status: Confirmed → In Progress
milestone: 1.99 → 2.1.8
Revision history for this message
Curtis Hovey (sinzui) wrote :

I landed the first branch for this feature that isolates all knowledge of the OpenID identity URL to the OpenIDPersistentIdentity object. Bug 237283 was fixed during the refactoring.

Curtis Hovey (sinzui)
Changed in launchpad:
milestone: 2.1.8 → 2.1.9
Revision history for this message
Curtis Hovey (sinzui) wrote :

More work was needed than planned to complete this feature. The principal work will be landed for testing next week,

Changed in launchpad-foundations:
milestone: 2.1.9 → 2.1.10
Revision history for this message
Curtis Hovey (sinzui) wrote :

Fix committed in RF 7139.

Changed in launchpad-foundations:
status: In Progress → Fix Committed
Revision history for this message
Curtis Hovey (sinzui) wrote :

The id vhost should now be available for QA. This vhost must be up for the release of this feature.

Revision history for this message
Curtis Hovey (sinzui) wrote :

Fix released in Launchpad 2.1.10.

Changed in launchpad-foundations:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.