Skip to content

Increase update download timeout - #287

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

Increase update download timeout#287
jai-deepsource merged 2 commits into
masterfrom
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-deepsourcejai-deepsource changed the title Increase timeout dlIncrease update download timeoutMar 25, 2026
@jai-deepsource
jai-deepsource merged commit 526ea0c into masterMar 25, 2026
2 of 5 checks passed
@jai-deepsource
jai-deepsource deleted the increase-timeout-dl branch March 25, 2026 19:36
@deepsource-io

deepsource-ioBot commented Mar 25, 2026

Copy link
Copy Markdown

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 GradeSecurity

Reliability

Complexity

Hygiene

Coverage

Code Review Summary

AnalyzerStatusUpdated (UTC)Details
GoMar 25, 2026 7:35p.m.Review ↗
SecretsMar 25, 2026 7:35p.m.Review ↗
Test coverageMar 25, 2026 7:35p.m.Review ↗

Code Coverage Summary

LanguageLine 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 freeto 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

@jai-deepsource@sourya-deepsource