Skip to content

Add in Okta plugin type.#26349

Merged
mdwn merged 2 commits intomasterfrom
mike.wilson/okta-plugin
May 17, 2023
Merged

Add in Okta plugin type.#26349
mdwn merged 2 commits intomasterfrom
mike.wilson/okta-plugin

Conversation

@mdwn
Copy link
Copy Markdown
Contributor

@mdwn mdwn commented May 16, 2023

The Okta plugin type has been added to support the hosted Okta plugin in enterprise.

The Okta plugin type has been added to support the hosted Okta plugin in
enterprise.
Comment thread api/types/plugin.go Outdated
Comment thread api/types/plugin.go Outdated
Comment thread api/types/plugin.go
if bearer == nil {
return trace.BadParameter("okta plugin must be used with the bearer token credential type")
}
if (bearer.Token == "") == (bearer.TokenFile == "") {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

huh, that's a clever way to do it 👍

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't take credit for it. :-) Copy paste job!

@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from justinas May 16, 2023 20:36
@mdwn mdwn enabled auto-merge May 17, 2023 01:59
@mdwn mdwn added this pull request to the merge queue May 17, 2023
Merged via the queue into master with commit 04891e5 May 17, 2023
@mdwn mdwn deleted the mike.wilson/okta-plugin branch May 17, 2023 02:29
@public-teleport-github-review-bot
Copy link
Copy Markdown

@mdwn See the table below for backport results.

Branch Result
branch/v13 Failed

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants