More External Auth validation, tests and some fixes#2426

Merged

mbarnes merged 32 commits intomainfrom

mociarain/external-auth-fixes

Aug 14, 2025

Commits on Aug 14, 2025

Bring work from PR-2245 over

mociarain
authored and
Matthew Barnes
committed
Add tests for URL

mociarain
authored and
Matthew Barnes
committed
Remove TODOs

mociarain
authored and
Matthew Barnes
committed
Add visibility tests

mociarain
authored and
Matthew Barnes
committed
Add PrefixPolicyType Validation

mociarain
authored and
Matthew Barnes
committed
Handle it correctly when passing to CS

mociarain
authored and
Matthew Barnes
committed
Fix test

mociarain
authored and
Matthew Barnes
committed
Add tests for BuildCSExternalAuth

mociarain
authored and
Matthew Barnes
committed
Add test for multiple clients

mociarain
authored and
Matthew Barnes
committed
Fix implementation of multiple clients

mociarain
authored and
Matthew Barnes
committed
Fix []ValidationRules builder logic

mociarain
authored and
Matthew Barnes
committed
Make CA and URL pointers

mociarain
authored and
Matthew Barnes
committed
Correctly parse Audiences

mociarain
authored and
Matthew Barnes
committed
Flesh out test

mociarain
authored and
Matthew Barnes
committed
Fix syntax

mociarain
authored and
Matthew Barnes
committed
Add comment

mociarain
authored and
Matthew Barnes
committed
Handle TokenIssuer correctly

mociarain
authored and
Matthew Barnes
committed
Add a zero test

mociarain
authored and
Matthew Barnes
committed
Add test for multiple Taints and fix it

mociarain
authored and
Matthew Barnes
committed
Add test for multiple ValidationRules

mociarain
authored and
Matthew Barnes
committed
Fix typo

mociarain
authored and
Matthew Barnes
committed
Tidy up

mociarain
authored and
Matthew Barnes
committed
Add extra assertion for NodePool create

mociarain
authored and
Matthew Barnes
committed
Move comment back

mociarain
authored and
Matthew Barnes
committed
Tidy imports

mociarain
authored and
Matthew Barnes
committed
Refactor and fix Claim generation

mociarain
authored and
Matthew Barnes
committed
Fix missing key name

mociarain
authored and
Matthew Barnes
committed
Fix incorrect json marshaller

mociarain
authored and
Matthew Barnes
committed
Revert Issuer pointer

mociarain
authored and
Matthew Barnes
committed
api: External auth token issuer URL will always be present
Matthew Barnes
committed
api: Add enum value UsernameClaimPrefixPolicyTypeNone
Matthew Barnes
committed
api: Pin autorest/go to 4.0.0-preview.73
Matthew Barnes
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

More External Auth validation, tests and some fixes#2426

More External Auth validation, tests and some fixes#2426
mbarnes merged 32 commits intomainfrom
mociarain/external-auth-fixes

Commits on Aug 14, 2025

Bring work from PR-2245 over

Add tests for URL

Remove TODOs

Add visibility tests

Add PrefixPolicyType Validation

Handle it correctly when passing to CS

Fix test

Add tests for BuildCSExternalAuth

Add test for multiple clients

Fix implementation of multiple clients

Fix []ValidationRules builder logic

Make CA and URL pointers

Correctly parse Audiences

Flesh out test

Fix syntax

Add comment

Handle TokenIssuer correctly

Add a zero test

Add test for multiple Taints and fix it

Add test for multiple ValidationRules

Fix typo

Tidy up

Add extra assertion for NodePool create

Move comment back

Tidy imports

Refactor and fix Claim generation

Fix missing key name

Fix incorrect json marshaller

Revert Issuer pointer

api: External auth token issuer URL will always be present

api: Add enum value UsernameClaimPrefixPolicyTypeNone

api: Pin autorest/go to 4.0.0-preview.73