Skip to content

Fetch: handle and fix HEAD request errors#678

Merged
nleush merged 6 commits into
devfrom
fix/cleanup-known-stream-errors
Jun 25, 2026
Merged

Fetch: handle and fix HEAD request errors#678
nleush merged 6 commits into
devfrom
fix/cleanup-known-stream-errors

Conversation

@nleush

@nleush nleush commented Jun 25, 2026

Copy link
Copy Markdown
Member

About the changes

  • Bug fix (non-breaking change which fixes an issue)

What has changed

Please describe what the code is doing (in this repo):

  • proper handlind body for HEAD requests

Checklist

  • I have performed a self-review of my code, including Files Changed view
  • Fix/feat branches in all related repos are named the same
  • All related PRs are submitted
  • Task is DONE and links back to this PR or no task

Review

Please note any risks, caveats, migration steps, release notices or additional tasks.

  • PRs have been tested on staging by DEV
  • PRs have been tested on staging by QA
  • PRs have been tested on staging by PRODUCT

Please describe the tests and give additional instructions if required, add screenshots and links if necessary.

@iparamonau iparamonau changed the title Handle HEAD requests error Fetch: handle and fix HEAD requests errors Jun 25, 2026
@iparamonau iparamonau changed the title Fetch: handle and fix HEAD requests errors Fetch: handle and fix HEAD request errors Jun 25, 2026
@nleush nleush force-pushed the fix/cleanup-known-stream-errors branch from d62a3fd to 138011b Compare June 25, 2026 15:54
@nleush nleush merged commit 9ebd356 into dev Jun 25, 2026
3 checks passed
@nleush nleush deleted the fix/cleanup-known-stream-errors branch June 25, 2026 17:47
nleush added a commit that referenced this pull request Jun 26, 2026
* reuse function `skipLoggingFetchError` to check not logged errors

* request: cleanup options.allowCache

* fetch: safe processing HEAD requests

* skip log for `ERR_STREAM_PREMATURE_CLOSE` error

* do not log ERR_HTTP2_STREAM_ERROR errors

* do not log ERR_HTTP2_SESSION_ERROR errors
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.

2 participants