Skip to content

#6299 Use native browser APIs for timeout promises - #6300

Merged
sosnovsky merged 5 commits into
masterfrom
6299-native-fetch
Sep 24, 2026
Merged

sosnovsky merged 5 commits into
masterfrom
6299-native-fetch

Conversation

@sosnovsky

@sosnovsky sosnovsky commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

This PR replaces manual timeout promises and response streaming with native browser APIs

close #6299


Tests (delete all except exactly one):

  • Does not need tests (refactor only, docs or internal changes)

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

@sosnovsky
sosnovsky requested a review from martgil September 22, 2026 19:14
@sosnovsky
sosnovsky marked this pull request as ready for review September 22, 2026 19:14
Comment thread extension/js/common/api/shared/api.ts Outdated
@sosnovsky
sosnovsky requested a review from martgil September 23, 2026 11:25

@martgil martgil left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM now 👍

@sosnovsky
sosnovsky merged commit cbed37f into master Sep 24, 2026
12 checks passed
@sosnovsky
sosnovsky deleted the 6299-native-fetch branch September 24, 2026 07:28
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.

Simplify fetch progress and timeout handling

2 participants