9
75%
develop: 75%

Ran 15 Mar 2019 05:36PM UTC

Files 210

Run time 10s

Badge

Embed ▾

Committed 15 Mar 2019 04:12PM UTC coverage: 11.949% (+0.07%) from 11.878%

Job # TESTING_PROFILE=unit-tests

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Don't save token from authorization header (#2228)


When Dockstore API is invoked with a Google token,
we look up the user based on the email associated with that
token. If we don't find a user, we create one. Then we would
save the token along with the user; this was
done to use that token to authenticate subsequent API calls to SAM.

Instead, don't persist the token to the database.

Added a transient field to User to hold the token and when
calls are made to SAM, check that field first.

Not wild about changing the User class to not directly map
to the database -- we don't use @Transient anywhere else. But this
seems to be the lowest impact way of implementing this.

Run Details

1733 of 14503 relevant lines covered (11.95%)

0.23 hits per line

ga4gh / dockstore / 7713 / 9
75%
develop: 75%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 7713.9 (TESTING_PROFILE=unit-tests)

ga4gh / dockstore / 7713 / 9 75% develop: 75%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 7713.9 (TESTING_PROFILE=unit-tests)

ga4gh / dockstore / 7713 / 9
75%
develop: 75%

README BADGES
x