DuendeSoftware / Duende.AccessTokenManagement

Automatic token management for machine to machine and user-centric web app OAuth and OIDC flows
Apache License 2.0
219 stars 63 forks source link

Remove unknown client error message #106

Closed josephdecock closed 4 months ago

josephdecock commented 4 months ago

The "unknown client" error message is sometimes confusing, because we're trying to infer that a client hasn't been configured by the absence of properties of that client. That isn't always true. We have new specific error messages that say both which property of the client we don't have and for which client, so let's just throw one of those errors instead.

@kallayj would this cover your use cases?

josephdecock commented 4 months ago

Related: https://github.com/DuendeSoftware/Duende.AccessTokenManagement/pull/73#issuecomment-2102724952

kallayj commented 4 months ago

Yes, this would address the paint point we experienced. Thanks.