-
Notifications
You must be signed in to change notification settings - Fork 5
Description
Current spec says:
The client metadata document MUST contain a client_id property whose value MUST compare and match the URL of the document using simple string comparison as defined in [RFC3986] Section 6.2.1.
I'm not sure of the reason this text is here, so perhaps there's a good reason?
There's definitely an argument against including client_id in the metadata document though. We've seen that implementations of https://datatracker.ietf.org/doc/html/rfc8414 often just blindly trust the issuer value in the metadata document, without verifying it matches the location they retrieved it from. Omitting client_id (by requiring it is not be present and must be ignored if present) from the metadata document sort of completely removes the possibility to do that.
The only downside I can think of is that if the metadata document is somehow getting exposed at multiple urls (https://example.com/client.json and https://www.example.com/client.json for example) then both would end up being valid client ids. I'm not sure if that's bad though?