Skip to content

Increase update download timeout#287

Merged
jai-deepsource merged 2 commits intomasterfrom
increase-timeout-dl
Mar 25, 2026
Merged

Increase update download timeout#287
jai-deepsource merged 2 commits intomasterfrom
increase-timeout-dl

Conversation

@jai-deepsource
Copy link
Copy Markdown
Contributor

No description provided.

- 30s was too short for downloading the ~15MB binary on slower connections
- Users were hitting "context deadline exceeded" during `deepsource update`
@jai-deepsource jai-deepsource changed the title Increase timeout dl Increase update download timeout Mar 25, 2026
@jai-deepsource jai-deepsource merged commit 526ea0c into master Mar 25, 2026
2 of 5 checks passed
@jai-deepsource jai-deepsource deleted the increase-timeout-dl branch March 25, 2026 19:36
@deepsource-io
Copy link
Copy Markdown

deepsource-io bot commented Mar 25, 2026

DeepSource Code Review

We reviewed changes in 5a2b74a...5916356 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Coverage  

Code Review Summary

Analyzer Status Updated (UTC) Details
Go Mar 25, 2026 7:35p.m. Review ↗
Secrets Mar 25, 2026 7:35p.m. Review ↗
Test coverage Mar 25, 2026 7:35p.m. Review ↗

Code Coverage Summary

Language Line Coverage (New Code) Line Coverage (Overall)
Aggregate
0%
[⤫ below threshold]
25.4%
Go
0%
[⤫ below threshold]
25.4%
[✓ above threshold]

➟ Additional coverage metrics may have been reported. See full coverage report ↗

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants