Skip to content

fix(net): ensure accurate content-length in response#749

Merged
j2rong4cn merged 5 commits intoOpenListTeam:mainfrom
pnparadise:main
Jul 19, 2025
Merged

fix(net): ensure accurate content-length in response#749
j2rong4cn merged 5 commits intoOpenListTeam:mainfrom
pnparadise:main

Conversation

@pnparadise
Copy link
Copy Markdown
Contributor

Chrome browsers were unable to preview thumbnails, reporting an 'ERR_HTTP_2_PROTOCOL_ERROR'. This was caused by an incorrect content-length header in the server's response for thumbnail images.

This commit corrects the content-length calculation, allowing Chrome and other compliant clients to render thumbnails correctly.
Fixes #729

Chrome browsers were unable to preview thumbnails, reporting an
'ERR_HTTP_2_PROTOCOL_ERROR'. This was caused by an incorrect
content-length header in the server's response for thumbnail images.

This commit corrects the content-length calculation, allowing
Chrome and other compliant clients to render thumbnails correctly.
@j2rong4cn j2rong4cn marked this pull request as draft July 18, 2025 09:59
@j2rong4cn j2rong4cn self-assigned this Jul 19, 2025
@j2rong4cn j2rong4cn changed the title fix(fs): ensure accurate content-length in http2 requests fix(net): ensure accurate content-length in response Jul 19, 2025
@j2rong4cn
Copy link
Copy Markdown
Member

我没测试,试一下有没有问题

@j2rong4cn j2rong4cn marked this pull request as ready for review July 19, 2025 10:48
@j2rong4cn j2rong4cn merged commit 86324d2 into OpenListTeam:main Jul 19, 2025
8 checks passed
abel533 pushed a commit to liuzh-fork/OpenList that referenced this pull request Jul 21, 2025
* fix(fs): ensure accurate content-length in http2 requests

Chrome browsers were unable to preview thumbnails, reporting an
'ERR_HTTP_2_PROTOCOL_ERROR'. This was caused by an incorrect
content-length header in the server's response for thumbnail images.

This commit corrects the content-length calculation, allowing
Chrome and other compliant clients to render thumbnails correctly.

* fix(net): ensure accurate content-length in response

* 补缺

* .

---------

Co-authored-by: zhiqiang.huang <[email protected]>
Co-authored-by: j2rong4cn <[email protected]>
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.

[BUG] 本机存储缩略图不能加载

2 participants