Skip to content

fix: increase jwt timeout for signed url cdn acceptance test#1240

Merged
itslenny merged 1 commit into
masterfrom
lenny/acceptance-test-jwt-timeout
Jul 16, 2026
Merged

fix: increase jwt timeout for signed url cdn acceptance test#1240
itslenny merged 1 commit into
masterfrom
lenny/acceptance-test-jwt-timeout

Conversation

@itslenny

Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?

Bug fix

What is the current behavior?

Singed URLs in acceptance tests expire in 7s

What is the new behavior?

10s jwt expiration

Additional context

Test flaky because jwt sometimes expires too quickly

@itslenny
itslenny requested a review from a team as a code owner July 16, 2026 19:50
@itslenny
itslenny enabled auto-merge (squash) July 16, 2026 19:52
@itslenny
itslenny merged commit e0684d9 into master Jul 16, 2026
22 checks passed
@itslenny
itslenny deleted the lenny/acceptance-test-jwt-timeout branch July 16, 2026 19:54
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 29529597431

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage decreased (-20.3%) to 59.141%

Details

  • Coverage decreased (-20.3%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 2389 coverage regressions across 100 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

2389 previously-covered lines in 100 files lost coverage.

Top 10 Files by Coverage Loss Lines Losing Coverage Coverage
src/http/routes/admin/pprof.ts 156 3.7%
src/storage/renderer/image.ts 156 31.2%
src/internal/monitoring/pprof/runtime.ts 130 0.98%
src/storage/protocols/iceberg/catalog/rest-catalog-client.ts 128 2.41%
src/internal/streams/hash-stream.ts 108 0.0%
src/internal/database/migrations/migrate.ts 104 37.91%
src/internal/monitoring/pprof/profile.ts 82 0.0%
src/storage/backend/file.ts 79 15.83%
src/internal/concurrency/async-abort-controller.ts 73 1.64%
src/internal/queue/queue.ts 71 1.33%

Coverage Stats

Coverage Status
Relevant Lines: 11879
Covered Lines: 7533
Line Coverage: 63.41%
Relevant Branches: 6915
Covered Branches: 3582
Branch Coverage: 51.8%
Branches in Coverage %: Yes
Coverage Strength: 388.12 hits per line

💛 - Coveralls

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants