Project

General

Profile

Actions

Defect #3780

closed

Attribute exchange with OpenID providers

Added by Gábor Molnár about 15 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
OpenID
Target version:
-
Start date:
2009-08-25
Due date:
% Done:

100%

Estimated time:
Resolution:
Wont fix
Affected version:

Description

Redmine should "include the attribute exchange parameters to authentication requests to access user information (such as name and email)" (quote: Google OpenID documentation), and i think this not what our OpenID implementation does now. As of r2847 , the attribute exchange doesnt work with Google accounts (this id the only provider i tried), and i think this is a major usability issue with the recent state of OpenID support.


Files

redmineOpenID.png (24 KB) redmineOpenID.png Ladislav Nesnera, 2010-11-29 12:15

Related issues

Related to Redmine - Defect #11778: openid : Fields not taken when logged in using Google account. Redmine 2.0.3Closed

Actions
Actions #1

Updated by Stas SUSHKOV about 15 years ago

The OpenID support as it is now is useless.
Please provide a way to define the required user details or enable only the bare basic openid support (just to allow a user to log in).

I would also rise the priority for this issue, though, hope it won't take months to be solved.

Thank you in advance.

Actions #2

Updated by Tom S about 15 years ago

+1

Actions #3

Updated by Stas SUSHKOV about 15 years ago

  • Status changed from New to Resolved

Here's a work around (dirty, lazy hack) which works for us with Launchpad's, WordPress's, Yahoo's OpenID.
Patch here: http://gist.github.com/179159

Actions #4

Updated by Jean-Philippe Lang almost 15 years ago

  • Assignee set to Eric Davis
Actions #5

Updated by Anonymous over 14 years ago

+1 -- the patch is about the same hack I made, but seeing something more robust built in would be nice.

Actions #6

Updated by Stas SUSHKOV over 14 years ago

Bump!
And soon we make an year.

Actions #7

Updated by Eric Davis over 14 years ago

  • Status changed from Resolved to New
  • Assignee deleted (Eric Davis)

Stas:

Redmine is developed by a group of volunteers. This hasn't been a high priority feature, which might be why no one has contributed their free time to build a more robust system for this.

Actions #8

Updated by Stas SUSHKOV over 14 years ago

Since this is important for me, I might consider spending some time/$ for coding/testing, if you could guide me what exactly is to be expected from existing patch/features making it a possible fix.

I agree with you and I know what volunteering means, but this was more like an ignorance.

Actions #9

Updated by Stas SUSHKOV over 14 years ago

Thank you.

Actions #10

Updated by M B about 14 years ago

+1

Actions #11

Updated by Ladislav Nesnera almost 14 years ago

Is there any progress in this case or I have some wrong settings? This doesn't look very well ;?(

Redmine 1.0.3.stable.4425 (MySQL)

Actions #12

Updated by Toshi MARUYAMA over 13 years ago

  • Category set to Accounts / authentication
Actions #13

Updated by Etienne Massip about 13 years ago

  • Category changed from Accounts / authentication to OpenID
Actions #14

Updated by Antoine Beaupré almost 13 years ago

  • % Done changed from 0 to 100

there is a fix for this: http://projects.andriylesyuk.com/projects/openid-fix

why isn't this factored in?!

Actions #15

Updated by Andrew Dahl almost 12 years ago

Antoine Beaupré wrote:

there is a fix for this: http://projects.andriylesyuk.com/projects/openid-fix

why isn't this factored in?!

I don't see the fix you refer to, Antoine. (Although, it could have been removed in the last year.) The only changes I see are in the repository and they involve allowing manual versus on-the-fly creation of users upon login, which Redmine provides currently and doesn't solve the problem this ticket identifies.

As this is relevant, I posted a patch here => http://www.redmine.org/issues/11778 <= that handles AX (Attributes Exchange) and works to auto-register users on Google and Yahoo (and probably others, but I only tested those two). I provide it more as an example of what needs to be done, but if it were to be pulled in, it does work better than the current implementation.

As well, if you like what you see and would like me to test & verify it across several OpenID providers (not just Google & Yahoo) and modify the code to include additional AX attributes as necessary, I can definitely do that and repost!

Actions #16

Updated by Antoine Beaupré over 11 years ago

Direct ref to an issue with a patch: #11778.

Actions #17

Updated by Go MAEDA almost 3 years ago

  • Status changed from New to Closed
  • Resolution set to Wont fix

The OpenID support has been dropped by #35755 for the upcoming Redmine 5.0.0.

Actions

Also available in: Atom PDF