1
71%
master: 71%

Ran 25 Mar 2020 10:57AM UTC

Files 67

Run time 7s

Badge

Embed ▾

Committed 25 Mar 2020 10:53AM UTC coverage: 86.777% (+0.2%) from 86.529%

Job # GO111MODULE=on

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

fix: handle concurrent  transactional errors in the refresh token grant handler (#402)

This commit provides the functionality required to address https://github.com/ory/hydra/issues/1719 & https://github.com/ory/hydra/issues/1735 by adding error checking to the RefreshTokenGrantHandler's PopulateTokenEndpointResponse method so it can deal with errors due to concurrent access. This will allow the authorization server to render a better error to the user-agent.

No longer returns fosite.ErrServerError in the event the storage. Instead a wrapped fosite.ErrNotFound is returned when fetching the refresh token fails due to it no longer being present. This scenario is caused when the user sends two or more request to refresh using the same token and one request gets into the handler just after the prior request finished and successfully committed its transaction.

Adds unit test coverage for transaction error handling logic added to the RefreshTokenGrantHandler's PopulateTokenEndpointResponse method

Run Details

3032 of 3494 relevant lines covered (86.78%)

107.69 hits per line

ory / fosite / 1457 / 1
71%
master: 71%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1457.1 (GO111MODULE=on)

ory / fosite / 1457 / 1 71% master: 71%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1457.1 (GO111MODULE=on)

ory / fosite / 1457 / 1
71%
master: 71%

README BADGES
x