Skip to content

fix: correctly tee thumbnail stream on upload#2068

Merged
pyphilia merged 9 commits intomainfrom
thumbnails-concurrent
Feb 10, 2026
Merged

fix: correctly tee thumbnail stream on upload#2068
pyphilia merged 9 commits intomainfrom
thumbnails-concurrent

Conversation

@pyphilia
Copy link
Contributor

@pyphilia pyphilia commented Feb 2, 2026

Make use of sharp.clone to avoid listening on the input file too many times. Sharp.clone allows to rely on the input file only once.

TODO

  • check tests

@pyphilia pyphilia self-assigned this Feb 2, 2026
@pyphilia pyphilia force-pushed the thumbnails-concurrent branch from 42c3f77 to e806cec Compare February 3, 2026 15:10
@pyphilia pyphilia requested a review from spaenleh February 5, 2026 10:31
@pyphilia pyphilia marked this pull request as ready for review February 5, 2026 10:31
Copy link
Member

@spaenleh spaenleh left a comment

Choose a reason for hiding this comment

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

Nice fix !

@sonarqubecloud
Copy link

@pyphilia pyphilia added this pull request to the merge queue Feb 10, 2026
Merged via the queue into main with commit 69ab32a Feb 10, 2026
13 of 14 checks passed
@pyphilia pyphilia deleted the thumbnails-concurrent branch February 10, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants