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

Discovery document contains string instead of array #33

Open
hzafar opened this issue May 21, 2021 · 3 comments
Open

Discovery document contains string instead of array #33

hzafar opened this issue May 21, 2021 · 3 comments

Comments

@hzafar
Copy link

hzafar commented May 21, 2021

The response from https://solidcommunity.net/.well-known/openid-configuration contains a string value for the token_endpoint_auth_methods_supported key, when this should be an array as per https://datatracker.ietf.org/doc/html/rfc8414.

@joachimvh

This comment has been minimized.

@RubenVerborgh RubenVerborgh transferred this issue from CommunitySolidServer/CommunitySolidServer May 25, 2021
@bourgeoa
Copy link
Member

@hzafar Thanks for reporting. How did you discover that, what are the implications ?

@hzafar
Copy link
Author

hzafar commented May 25, 2021

Hi @bourgeoa, this was discovered when configuring a Solid OIDC provider for a Spring application using the Spring Security framework. The framework usually allows auto-configuration of an OIDC provider from its discovery document, but in this case it's unable to parse the document due to the type mismatch on token_endpoint_auth_methods_supported. As a result, some custom classes have been written to workaround the issue (so this isn't an immediate blocker), but it would be ideal if the config-only approach could be used for adding https://solidcommunity.net as an identity provider for Spring applications.

@bourgeoa bourgeoa transferred this issue from nodeSolidServer/node-solid-server Jun 1, 2021
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

No branches or pull requests

4 participants