fix: preserve responsive images in proxied and reading views - #1231
Merged
marcomarcogd merged 1 commit intoSep 28, 2026
Merged
Conversation
marcomarcogd
marked this pull request as ready for review
September 28, 2026 02:42
Member
|
@marcomarcogd pr 尽量不要直接往 main 里面合并,这样的话 release 管理不太好做。tools 文件夹下有个用于新版本的脚本:new-release.ps1,执行后可以创建一个新的 release 分支,所有 pr 合并进这个分支,然后攒一段时间统一发新的 release。 |
marcomarcogd
added a commit
that referenced
this pull request
Sep 29, 2026
marcomarcogd
added a commit
that referenced
this pull request
Sep 29, 2026
revert:将 #1231 图片修复迁至发布分支
This was referenced Sep 29, 2026
marcomarcogd
added a commit
that referenced
this pull request
Sep 29, 2026
marcomarcogd
added a commit
that referenced
this pull request
Sep 29, 2026
…ink-upstream fix:恢复 #1231 并补齐论坛图片与防盗链处理
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The webpage proxy rewrites
srcbut leaves responsive image candidates insrcsetpointing at the app origin. Readability can also detect the page lead image from metadata while returning article HTML with no images.This change:
imgandsourcesrcset/data-srcsetattributes while preserving width and density descriptors;The existing media and full-text test files now cover the Next.js
<picture>structure from the report, mixed-casesrcSet, entity decoding, candidate descriptors, lead-image restoration, and duplicate-image protection.Validation:
go test -count=1 ./internal/handlers/media ./internal/handlers/corego vet ./...go build ./...npm run test:unit(247 tests)npx eslint .(0 errors; existing warnings remain)npm run buildgit diff --checkThe full Go suite's existing macOS-only AppImage and storage migration failures reproduce unchanged on
upstream/main; the affected packages and Linux CI remain the authoritative checks for this patch.Fixes #1230