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.

Sign up for GitHub

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

Jump to bottom

Don't include PAR in discovery if feature is not enabled in license #1580

Open
josephdecock opened this issue Jul 29, 2024 · 1 comment
Open

Don't include PAR in discovery if feature is not enabled in license #1580

josephdecock opened this issue Jul 29, 2024 · 1 comment
Labels
enhancement New feature or request
Milestone
7.2.0

Comments

@josephdecock
Copy link
Member

Including PAR in the disco doc when the license doesn't allow it can confuse clients that will then attempt to use PAR and fail. E.g., IdentityModel.OidcClient will try to push if the PAR endpoint is present.
@josephdecock josephdecock mentioned this issue Jul 31, 2024
Closed
@brockallen
Copy link
Member

Should this (and many other features) be more license driven, rather than options driven?

@brockallen brockallen mentioned this issue Sep 17, 2024
Open
@brockallen brockallen added this to the 7.2.0 milestone Sep 17, 2024
@brockallen brockallen added the enhancement New feature or request label Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
7.2.0
Development

No branches or pull requests
2 participants
@brockallen @josephdecock