microlinkhq / html-get / 41037864662

97%

push

github

fix(fetch): inherit the URL fragment across redirects (#309)

* fix(fetch): inherit the URL fragment across redirects

Prerender keeps the fragment since Puppeteer v24, which appends
urlFragment to request.url() and response.url().

Fetch mode dropped it when a redirect Location had no fragment, so
/one#hello -> /target returned /target while a browser lands on
/target#hello. RFC 9110 section 10.2.2 requires inheriting it, and an
empty fragment in Location counts as present, hence the href check.

The fallback title no longer includes the fragment.

Closes #209

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* fix(fetch): ignore a falsy beforeRedirect hook

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* test(fetch): cover got hooks kept across fragment inheritance

Replacing beforeRedirect still copies the other got hooks. Lock init,
beforeRequest, beforeRetry, afterResponse, and beforeError so dropping
one fails, and note why an empty fragment must not be inherited.

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

649 of 715 branches covered (90.77%)

Branch coverage included in aggregate %.

11 of 11 new or added lines in 2 files covered. (100.0%)

2030 of 2061 relevant lines covered (98.5%)

164.45 hits per line