fix: Bulk export fails due to S3 upload minimal version#10782
Merged
mergify[bot] merged 2 commits intomasterfrom Feb 19, 2026
Merged
fix: Bulk export fails due to S3 upload minimal version#10782mergify[bot] merged 2 commits intomasterfrom
mergify[bot] merged 2 commits intomasterfrom
Conversation
Contributor
|
#10775 が不要なのであれば close してください |
yuki-takei
requested changes
Feb 16, 2026
...tures/page-bulk-export/server/service/page-bulk-export-job-cron/steps/compress-and-upload.ts
Show resolved
Hide resolved
Contributor
|
@mergify queue |
Contributor
Merge Queue StatusRule:
This pull request spent 13 minutes 20 seconds in the queue, including 13 minutes 7 seconds running CI. Required conditions to merge
|
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.
1段階目のPRのレビューを反映したうえで、修正を必要最小限に変更したため、新たにPRを作成しました。
また、RCリリースにより本番環境(Inner Wiki)での動作を既に確認しています。
Task
https://redmine.weseek.co.jp/issues/178419
[Bulk Export] pdf-converter が AWS との組み合わせで発生している不具合を修正する
症状
AWS 環境で bulk export(Markdown/PDF 両方)が失敗し、ダウンロード時に NoSuchKey エラーが発生していた。GCS 環境では正常に動作していた。
バグの原因
PDFエクスポート>フロー概要図ステップ14に続く、「PDFをストレージにアップロード(tar.gz化)」で、tar.gz圧縮データを生成する archiver のストリームを AWS SDK に渡した時点で型チェックに失敗し、データが一切流れないまま即座にエラーになっていた。
これは、archiver と AWS SDK が異なる Readable クラスを参照しているため、AWS SDK が archiver のストリームをストリームとして認識できていなかったことによる。GCS の実装はこの影響を受けないため、GCS 環境では正常に動作する。
変更点
バグ対応
compress-and-upload.ts: archiver ストリームを PassThrough でラップarchiver のストリームを、AWS SDK が認識できる Node.js ネイティブのストリーム(PassThrough)に中継させることで、instanceof Readable チェックを通過させた。
また、archiver のエラーを uploadStream に伝播させるハンドリングと、cleanup 時に uploadStream も破棄されるよう setStreamsInExecution への登録を追加した。