Skip to content

Introduce group and app name Okta import rule regexes.#26561

Merged
mdwn merged 2 commits intomasterfrom
mike.wilson/okta-import-rules-regex
May 23, 2023
Merged

Introduce group and app name Okta import rule regexes.#26561
mdwn merged 2 commits intomasterfrom
mike.wilson/okta-import-rules-regex

Conversation

@mdwn
Copy link
Copy Markdown
Contributor

@mdwn mdwn commented May 19, 2023

Okta import rules now support regex name matching. This will allow for matching any group or application which conforms to a given regex, which will make generating import rule matches less verbose as users won't have to list out groups and applications by ID.

Validation of regexes happens at create and update time. This is to avoid compilation of regexes every time an Okta Import Rule unmarshaled. The Okta service will compile these regexes as needed.

@github-actions github-actions Bot requested review from avatus and zmb3 May 19, 2023 02:11
Comment thread api/proto/teleport/legacy/types/types.proto Outdated
Mike Wilson added 2 commits May 23, 2023 13:42
Okta import rules now support regex name matching. This will allow for
matching any group or application which conforms to a given regex, which
will make generating import rule matches less verbose as users won't have
to list out groups and applications by ID.

Validation of regexes happens at create and update time. This is to avoid
compilation of regexes every time an Okta Import Rule unmarshaled. The Okta
service will compile these regexes as needed.
@mdwn mdwn force-pushed the mike.wilson/okta-import-rules-regex branch from 96dd327 to cba1e33 Compare May 23, 2023 17:43
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from zmb3 May 23, 2023 17:44
@mdwn mdwn added this pull request to the merge queue May 23, 2023
Merged via the queue into master with commit fe6e044 May 23, 2023
@mdwn mdwn deleted the mike.wilson/okta-import-rules-regex branch May 23, 2023 18:49
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