Skip to content

fix: Bulk export fails due to S3 upload minimal version#10782

Merged
mergify[bot] merged 2 commits intomasterfrom
fix/178290-bulk-export-aws-minimal
Feb 19, 2026
Merged

fix: Bulk export fails due to S3 upload minimal version#10782
mergify[bot] merged 2 commits intomasterfrom
fix/178290-bulk-export-aws-minimal

Conversation

@ryotaro-nagahara
Copy link

@ryotaro-nagahara ryotaro-nagahara commented Feb 13, 2026

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 環境では正常に動作する。

変更点

バグ対応

  1. compress-and-upload.ts: archiver ストリームを PassThrough でラップ
    archiver のストリームを、AWS SDK が認識できる Node.js ネイティブのストリーム(PassThrough)に中継させることで、instanceof Readable チェックを通過させた。
    また、archiver のエラーを uploadStream に伝播させるハンドリングと、cleanup 時に uploadStream も破棄されるよう setStreamsInExecution への登録を追加した。

@yuki-takei
Copy link
Contributor

#10775 が不要なのであれば close してください
ブランチも不要なら削除してください

@yuki-takei yuki-takei changed the title fix: bulk export fails due to S3 upload minimal version fix: Bulk export fails due to S3 upload minimal version Feb 19, 2026
@yuki-takei
Copy link
Contributor

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Feb 19, 2026

Merge Queue Status

Rule: default


This pull request spent 13 minutes 20 seconds in the queue, including 13 minutes 7 seconds running CI.

Required conditions to merge
  • -check-failure ~= ci-app-
  • -check-failure ~= ci-slackbot-
  • -check-failure ~= test-prod-node20 /
  • check-success = test-prod-node20 / build-prod
  • check-success ~= ci-app-launch-dev
  • check-success ~= ci-app-lint
  • check-success ~= ci-app-test
  • check-success ~= test-prod-node20 / launch-prod
  • check-success ~= test-prod-node20 / run-playwright

@mergify mergify bot added the queued label Feb 19, 2026
mergify bot added a commit that referenced this pull request Feb 19, 2026
@mergify mergify bot merged commit a1d2dca into master Feb 19, 2026
29 checks passed
@mergify mergify bot deleted the fix/178290-bulk-export-aws-minimal branch February 19, 2026 13:29
@mergify mergify bot removed the queued label Feb 19, 2026
@github-actions github-actions bot mentioned this pull request Feb 19, 2026
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