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

[Feature Request] Move AadIssuerValidator (and co) to its own library #1487

Closed
jmprieur opened this issue Oct 15, 2021 · 1 comment
Closed
Labels
enhancement New feature or request
Milestone

Comments

@jmprieur
Copy link
Collaborator

jmprieur commented Oct 15, 2021

Is your feature request related to a problem? Please describe.
AadIssuerValidator is usable in many applications that need to validate tokens, not only in Microsoft.Identity.Web. In particular this is needed in SAL and MISE. Ideally it should be in Wilson, but this needs to be discussed and planned.

Describe the solution you'd like

  • Extract AadIssuerValidator and MicrosoftIdentityWebIssuerValidator to a new assembly
    • Microsoft.Identity.Web.AadIssuerValidator or Microsoft.IdentityModel.Aad?
  • allow minimum dependencies, which means we might want to remove the Options.

Describe alternatives you've considered
Force all applications to depend on IdWeb, but MISE and SAL won't.

@jmprieur jmprieur added the enhancement New feature or request label Oct 15, 2021
@jennyf19 jennyf19 added this to the 1.20 milestone Oct 28, 2021
@jennyf19 jennyf19 modified the milestones: 1.20, 1.19 Nov 1, 2021
@jennyf19
Copy link
Collaborator

jennyf19 commented Nov 2, 2021

@jennyf19 jennyf19 closed this as completed Nov 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants