search

thephpleague / oauth2-server

A spec compliant, secure by default PHP OAuth 2.0 Server
https://oauth2.thephpleague.com
MIT License
6.51k stars 1.12k forks source link

Added exception chaining in BearerTokenValidator #1243

Closed rhertogh closed 1 year ago

rhertogh commented 3 years ago

Added exception chaining for RequiredConstraintsViolated in \League\OAuth2\Server\AuthorizationValidators\BearerTokenValidator::validateAuthorization().

This helps with logging the exception (similar to line 103).

annervisser commented 1 year ago

We're also running into how hard these issues are to debug. This PR would greatly help with that

dungsaga commented 1 year ago

oops, I created an identical PR (#1335) for this problem. We should either merge this or #1335.

Sephster commented 1 year ago

Thanks for the PR. I will merge this in tomorrow and close #1335 - thanks to both @rhertogh and @dungsaga

dungsaga commented 1 year ago

@Sephster let's merge this PR

Sephster commented 1 year ago

Merged. Many thanks to all that flagged or worked on this