7713
75%

Ran 15 Mar 2019 05:36PM UTC

Jobs 5

Files 212

Run time 41min

Badge

Embed ▾

pending completion

Build # 7713

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

9631 of 14643 relevant lines covered (65.77%)

1.8 hits per line

Jobs

ID	Job ID	Ran	Coverage
9	7713.9 (TESTING_PROFILE=unit-tests)	15 Mar 2019 05:36PM UTC	11.95	Travis Job 7713.9
10	7713.10 (TESTING_PROFILE=integration-tests)	15 Mar 2019 06:10PM UTC	12.52	Travis Job 7713.10
11	7713.11 (TESTING_PROFILE=tool-integration-tests)	15 Mar 2019 06:08PM UTC	41.97	Travis Job 7713.11
12	7713.12 (TESTING_PROFILE=workflow-integration-tests)	15 Mar 2019 06:15PM UTC	42.2	Travis Job 7713.12
13	7713.13 (TESTING_PROFILE=other-integration-tests)	15 Mar 2019 06:17PM UTC	47.38	Travis Job 7713.13

Source Files on build 7713

Detailed source file information is not available for this build.

ga4gh / dockstore / 7713
75%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 7713

ga4gh / dockstore / 7713 75%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 7713

ga4gh / dockstore / 7713
75%

README BADGES
x