Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Any primaryKey other than UID doesn't work #436

Closed
mzico opened this issue Jan 14, 2017 · 4 comments
Closed

Any primaryKey other than UID doesn't work #436

mzico opened this issue Jan 14, 2017 · 4 comments
Assignees
@qbert2k
Labels
bug bug in code exclude from release notes issues found during QA that should be excluded from release notes
Milestone
CE 3.0.0

Comments

@mzico
Copy link
Contributor

mzico commented Jan 14, 2017

  • Issue: If we want to use 'email_address' or any other attribute other than 'UID' as login attribute; it doesn't work.

  • Stack trace: http://pastebin.com/sBieYYH3

  • oxAuth version: 2.4.4.sp2_openldap / Implementation-Build: b0a8766

  • How to reproduce:

  • Checked with one IDP which is using a remote LDAP which is storing user information.
  • 'Manage Authentication'
    • Primary Key: email [ or any other attribute but uid / sAMAccountName ]
    • Local Primary Key: email [ any attribute other than UID ]
@mzico mzico added the bug bug in code label Jan 14, 2017
@nynymike nynymike added this to the CE 3.0.0 milestone Jan 14, 2017
@yurem yurem self-assigned this Jan 15, 2017
@yurem
Copy link
Contributor

yurem commented Jan 18, 2017

Fixed in CE 3.0.0 and in 2.4.4 SP3. New builds in maven contains this fix

@yurem yurem modified the milestones: CE 3.0.0 QA, CE 3.0.0 Jan 18, 2017
@mzico
Copy link
Contributor Author

mzico commented Jan 18, 2017

@yurem : Thanks; reassigning the issue to Zamil for quick testing.

@yurem yurem modified the milestones: CE 3.0.0, CE 3.0.0 QA Jan 22, 2017
@yurem yurem added the exclude from release notes issues found during QA that should be excluded from release notes label Jan 22, 2017
@yurem
Copy link
Contributor

yurem commented Jan 23, 2017

@zamilskhan can you retest it?

@nynymike nynymike assigned qbert2k and unassigned yurem and zamilskhan Jan 23, 2017
@zamilskhan
Copy link

I am on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@qbert2k qbert2k

Labels
bug bug in code exclude from release notes issues found during QA that should be excluded from release notes
Projects
None yet
Milestone
CE 3.0.0
Development

No branches or pull requests
5 participants
@qbert2k @nynymike @yurem @mzico @zamilskhan