Feature #3892

Site should support Open ID authentication

Added by James Turnbull almost 2 years ago. Updated 2 months ago.

Status:In Topic Branch Pending Review Start date:05/27/2010
Priority:Normal Due date:
Assignee:Kelsey Hightower % Done:

0%

Category:module site Spent time: -
Target version:-
Keywords: Affected URL:
Branch:http://github.com/igal/puppet-dashboard/commits/with_devise_openid
Votes: 3

History

Updated by Igal Koshevoy almost 2 years ago

  • Category set to module site
  • Status changed from Unreviewed to Accepted
  • Assignee set to Igal Koshevoy
  • Priority changed from Normal to High

I <3 OpenID and having it would be a good fit for this site.

Updated by Igal Koshevoy almost 2 years ago

  • Priority changed from High to Low

I’m reducing this to low priority because: 1. The current login works. 2. The app is uses the Devise authentication system, which does not provide a trivial way to add OpenID authentication. 3. Replacing Devise with another authentication system, like Authlogic, would take a day or two to do, which may not be the highest priority use of this much time.

Updated by Paul Lathrop almost 2 years ago

Igal Koshevoy wrote:

I’m reducing this to low priority because: 1. The current login works. 2. The app is uses the Devise authentication system, which does not provide a trivial way to add OpenID authentication. 3. Replacing Devise with another authentication system, like Authlogic, would take a day or two to do, which may not be the highest priority use of this much time.

Well, it is up to you guys to set priorities, but I’m not creating Yet Another Login To Keep Track Of.

So, until you support OpenID, I for one will not be populating the Forge with that great content Michael DeHaan mentioned.

Updated by Igal Koshevoy almost 2 years ago

  • Status changed from Accepted to Needs Decision
  • Assignee changed from Igal Koshevoy to Anonymous

Paul: Thank you for the honest feedback. I’ve gotten some more like this elsewhere too.

Michael (Product Manager): People keep saying they want OpenID and so would I, because it’s easy to use and will improve Forge adoption. Unfortunately, making it possible for people to login via OpenID or password will require 1-2 days of work. If you want to do this, how do you want to prioritize and schedule this work?

The work involved is: 1. Tear out the existing “Devise” authentication system, because no one’s been able to figure out how to add OpenID to it — and a number of people have tried over the last 6 months. 2. Add a new authentication system that supports the ability to login using either a password or OpenID, either using “Authlogic” or “restful-authentication”. 3. Convert all the code and data to support the new system, so that people with existing accounts and passwords can still login.

Updated by Igal Koshevoy almost 2 years ago

  • Tracker changed from Bug to Feature

Updated by Igal Koshevoy almost 2 years ago

  • Status changed from Needs Decision to Accepted
  • Assignee changed from Anonymous to Igal Koshevoy
  • Target version set to 54

Updated by Igal Koshevoy almost 2 years ago

  • Target version changed from 54 to 55

I have hybrid OpenID and username-password authentication working in development. Am putting the finishing touches on and hope to release this soon.

Updated by Igal Koshevoy almost 2 years ago

  • Assignee deleted (Igal Koshevoy)

Updated by Igal Koshevoy over 1 year ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Priority changed from Low to Normal
  • Branch set to http://github.com/igal/puppet-dashboard/commits/with_devise_openid

I don’t think this code is done yet, but it’s very close. I haven’t been had time to work on it for months and thus have no idea what’s left to do. At this point, it needs to be carefully reviewed to determine whether it’s really done or what else needs to be addressed.

Updated by Anonymous over 1 year ago

  • Target version changed from 55 to 73

Updated by Nigel Kersten 4 months ago

  • Assignee set to Kelsey Hightower

Kelsey, we need someone to see if this code is even vaguely functional right now.

If not, we should just close this and deal with re-inventing auth later.

Updated by Daniel Pittman 2 months ago

  • Target version deleted (73)

Also available in: Atom PDF