ory / hydra

OpenID Certified™ OpenID Connect and OAuth Provider written in Go - cloud native, security-first, open source API security for your infrastructure. SDKs for any language. Works with Hardware Security Modules. Compatible with MITREid.
https://www.ory.sh/hydra/?utm_source=github&utm_medium=banner&utm_campaign=hydra
Apache License 2.0
15.27k stars 1.47k forks source link

fix: wrap token handler in transaction #3730

Closed hperl closed 4 months ago

hperl commented 4 months ago

Fixes https://github.com/ory-corp/cloud/issues/5987

Related issue(s)

Checklist

Further Comments

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 87.75510% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 76.31%. Comparing base (57096be) to head (e2d4900). Report is 4 commits behind head on master.

Files Patch % Lines
oauth2/handler.go 88.23% 2 Missing and 2 partials :warning:
persistence/sql/persister.go 66.66% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #3730 +/- ## ========================================== + Coverage 76.22% 76.31% +0.08% ========================================== Files 134 134 Lines 10210 10212 +2 ========================================== + Hits 7783 7793 +10 + Misses 1910 1902 -8 Partials 517 517 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.