- 🇳🇱Netherlands bojan_dev
Logging has been improved in: 📌 Reduce logging severity/don't log expired tokens/401s Needs review
Often is the debugging of oAuth2 errror messages a pain. Luckly the OAuthServerException class implements a hint, which helps finding the issue with failing oAuth2 requests.
Unfortunatly this hint is not saved to watchdog. I made a patch which also logs the hint:
Before:
League\OAuth2\Server\Exception\OAuthServerException: The request is missing a required parameter, includes an invalid parameter value, includes a parameter more than once, or is otherwise malformed. in League\OAuth2\Server\Exception\OAuthServerException::invalidRequest() (line 126 of /home/lessmess/domains/stage.lessmess.io/www/vendor/league/oauth2-server/src/Exception/OAuthServerException.php).
After:
League\OAuth2\Server\Exception\OAuthServerException: The request is missing a required parameter, includes an invalid parameter value, includes a parameter more than once, or is otherwise malformed. in League\OAuth2\Server\Exception\OAuthServerException::invalidRequest() (line 126 of /home/lessmess/domains/stage.lessmess.io/www/vendor/league/oauth2-server/src/Exception/OAuthServerException.php). Hint: Authorization code has been revoked
I hope this small improvement can be reviewed and commited by a maintainer.
Closed: duplicate
5.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Logging has been improved in: 📌 Reduce logging severity/don't log expired tokens/401s Needs review