Skip to content

Fix import with optional identity attributes #36887

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

Merged
merged 4 commits into from
Apr 16, 2025
Merged

Conversation

dbanck
Copy link
Member

@dbanck dbanck commented Apr 15, 2025

This PR addresses two issues when using optional attributes in an identity. We now correctly use the fully populated identity from the provider instead of the one from config.

Target Release

1.12.x

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

@dbanck dbanck added bug no-changelog-needed Add this to your PR if the change does not require a changelog entry labels Apr 15, 2025
@dbanck dbanck force-pushed the dbanck/fix-identity-decode branch from 2138bd7 to 1ac4d90 Compare April 15, 2025 16:50
@dbanck dbanck marked this pull request as ready for review April 16, 2025 07:30
@dbanck dbanck requested a review from a team as a code owner April 16, 2025 07:30
@dbanck dbanck added the 1.12-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged label Apr 16, 2025
@dbanck dbanck merged commit 7414a3f into main Apr 16, 2025
15 of 16 checks passed
@dbanck dbanck deleted the dbanck/fix-identity-decode branch April 16, 2025 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.12-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged bug no-changelog-needed Add this to your PR if the change does not require a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants