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

Extensibility tests: Audience #2861

Merged
merged 8 commits into from
Oct 17, 2024
Merged

Conversation

iNinja
Copy link
Contributor

@iNinja iNinja commented Oct 2, 2024

Extensibility tests: Audience

  • Added special exception to ValidationError to handle unknown exception types
  • Added new LogMessage for the case above
  • Added extensibility tests for JWT audience validation

Part of #2711

@iNinja iNinja requested a review from a team as a code owner October 2, 2024 14:29
@iNinja iNinja marked this pull request as draft October 3, 2024 16:49
@iNinja iNinja marked this pull request as ready for review October 16, 2024 13:55
@@ -15,6 +15,8 @@ internal static class LogMessages
// general
public const string IDX10000 = "IDX10000: The parameter '{0}' cannot be a 'null' or an empty object. ";
public const string IDX10001 = "IDX10001: Invalid argument '{0}'. Argument must be of type '{1}'.";
// TODO: Confirm the wording of this message. Possibly create a wiki page to refer users to for more information.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let make sure we have a tracking GitHub issue, otherwise we might forget. agree on an aka.ms link

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

@jennyf19 jennyf19 left a comment

Choose a reason for hiding this comment

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

I think this looks great.
:shipit:

@iNinja iNinja linked an issue Oct 17, 2024 that may be closed by this pull request
# Conflicts:
#	src/Microsoft.IdentityModel.Tokens/InternalAPI.Unshipped.txt
#	src/Microsoft.IdentityModel.Tokens/Validation/Results/Details/AudienceValidationError.cs
Copy link
Contributor

@FuPingFranco FuPingFranco left a comment

Choose a reason for hiding this comment

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

LGTM

@iNinja iNinja merged commit d88aa8d into dev Oct 17, 2024
6 checks passed
@iNinja iNinja deleted the iinglese/extensibility-tests-audience branch November 15, 2024 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add extensibility tests for Audience validation in JsonWebTokenHandler
3 participants