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.

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

fix(ui): pick email and name based on claim values at time of login #17626

Merged
merged 13 commits into from
Aug 30, 2024

Conversation

chirag-madlani
Copy link
Collaborator

@chirag-madlani chirag-madlani commented Aug 29, 2024

Describe your changes:

Fixes: Azure login issue with preferred_username as claim for initial admins

  • Scope as preferred_username first
362586579-36834193-3007-437b-b9c2-5b1c2a0c89ca.mov
  • Scope as email first
362586523-9d8bd397-7932-4478-8c49-abbe29276910.mov
  • Error upon user present but having miss-match in details extracted from token
    image

I worked on ... because ...

Type of change:

  • Bug fix
  • Improvement
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Checklist:

  • I have read the CONTRIBUTING document.
  • My PR title is Fixes <issue-number>: <short explanation>
  • I have commented on my code, particularly in hard-to-understand areas.
  • For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Copy link
Contributor

github-actions bot commented Aug 29, 2024

Jest test Coverage

UI tests summary

Lines Statements Branches Functions
Coverage: 65%
65.11% (37160/57072) 41.6% (14774/35517) 43.93% (4525/10300)

karanh37
karanh37 previously approved these changes Aug 29, 2024
@chirag-madlani chirag-madlani added the e2e This will trigger e2e test workflows label Aug 29, 2024
Copy link

Copy link

@chirag-madlani chirag-madlani enabled auto-merge (squash) August 30, 2024 05:48
@chirag-madlani chirag-madlani merged commit fa3529f into main Aug 30, 2024
28 checks passed
@chirag-madlani chirag-madlani deleted the map-claim-login branch August 30, 2024 06:27
chirag-madlani added a commit that referenced this pull request Aug 30, 2024
…17626)

* fix(ui): pick email and name based on claim values at time of login

* Update Error Message

* Update code for better message

* Typo

* fix playwright failures

* fix playwright tests

* fix tests

* fix login spec failing

---------

Co-authored-by: mohitdeuex <[email protected]>
(cherry picked from commit fa3529f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e This will trigger e2e test workflows safe to test Add this label to run secure Github workflows on PRs UI UI specific issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants