Skip to content
This repository was archived by the owner on Dec 14, 2025. It is now read-only.

fix: use 3s timeout rather than 15 default#237

Merged
bcoe merged 2 commits intomasterfrom
timeout-strategy
Sep 13, 2019
Merged

fix: use 3s timeout rather than 15 default#237
bcoe merged 2 commits intomasterfrom
timeout-strategy

Conversation

@bcoe
Copy link

@bcoe bcoe commented Sep 13, 2019

I've tested with GOOGLE_APPLICATION_CREDENTIALS set, and gcloud logged in, we only hit the metadata server IP if these authentication strategies are not found, in which case we will now timeout after 3s rather than 15s.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 13, 2019
@codecov
Copy link

codecov bot commented Sep 13, 2019

Codecov Report

Merging #237 into master will increase coverage by 0.08%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #237      +/-   ##
==========================================
+ Coverage   89.25%   89.34%   +0.08%     
==========================================
  Files           1        1              
  Lines         121      122       +1     
  Branches       28       28              
==========================================
+ Hits          108      109       +1     
  Misses         12       12              
  Partials        1        1
Impacted Files Coverage Δ
src/index.ts 89.34% <100%> (+0.08%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d04207b...d52e590. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Sep 13, 2019

Codecov Report

Merging #237 into master will increase coverage by 0.08%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #237      +/-   ##
==========================================
+ Coverage   89.25%   89.34%   +0.08%     
==========================================
  Files           1        1              
  Lines         121      122       +1     
  Branches       28       28              
==========================================
+ Hits          108      109       +1     
  Misses         12       12              
  Partials        1        1
Impacted Files Coverage Δ
src/index.ts 89.34% <100%> (+0.08%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d04207b...d52e590. Read the comment docs.

@bcoe bcoe merged commit 231ca5c into master Sep 13, 2019
@bcoe bcoe deleted the timeout-strategy branch September 13, 2019 19:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants