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

[v17] [entraid] store entra appId in plugin settings #48798

Merged
merged 2 commits into from
Nov 12, 2024

Conversation

tigrato
Copy link
Contributor

@tigrato tigrato commented Nov 12, 2024

Backport #48754 to branch/v17

This PR stores the Entra ID appID of the application used for SSO in Entra ID plugin settings.
This field filled in a best effort scenario and might be empty for old Entra Plugins.
Newly created plugins will make the field mandatory.

Signed-off-by: Tiago Silva <[email protected]>
@tigrato tigrato added the no-changelog Indicates that a PR does not require a changelog entry label Nov 12, 2024
@github-actions github-actions bot added backport size/sm tctl tctl - Teleport admin tool labels Nov 12, 2024
@tigrato tigrato added this pull request to the merge queue Nov 12, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 12, 2024
@tigrato tigrato added this pull request to the merge queue Nov 12, 2024
Merged via the queue into branch/v17 with commit bd3cfc7 Nov 12, 2024
44 of 45 checks passed
@tigrato tigrato deleted the bot/backport-48754-branch/v17 branch November 12, 2024 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport no-changelog Indicates that a PR does not require a changelog entry size/sm tctl tctl - Teleport admin tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants