Merge branch 'develop' of github.com:shiguredo/sora-archive-uploader …

…into feature/add-recording-metadata-config

.github/workflows/ci.yaml

@@ -9,7 +9,7 @@ on:
 jobs:
   build:
     name: ci
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     steps:
       - uses: actions/checkout@v4
       - name: setup go

.github/workflows/release.yml

@@ -7,7 +7,7 @@ on:
 
 jobs:
   release:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     steps:
       - uses: actions/checkout@v4
 

CHANGES.md

@@ -14,6 +14,9 @@
 - [UPDATE] 設定に `exclude_webhook_recording_metadata` を追加し、report ファイルアップロード後のウェブフックに `recording_metadata` を含めるかどうか設定できるようにする
   - デフォルトは `false` で `recording_metadata` を送信するウェブフックに含める
   - `true` を設定するとレポートファイルに `recording_metadata` または `metadata` が含まれていてもウェブフックには含めない
+- [FIX] 5GB を超えるファイルのアップロード時に帯域制限がかかるように修正する
+  - 帯域制限設定を行ってもマルチパートアップロードを有効にし、マルチパートアップロードの並列アップロード数を 1 つずつにすることで帯域制限を行う
+  - この修正以前は、帯域制限設定を行うとマルチパートアップロードが無効となり 5GB を超えるファイルのアップロードができなかった
   - @tnamao
 - [UPDATE] report ファイルアップロード後のウェブフックに `recording_metadata` を追加する
   - アップロードした report ファイルの `recording_metadata` または `metadata` の内容をウェブフックの `recording_metadata` に含めて送信する
@@ -22,6 +25,11 @@
     - ウェブフックに含める際のキーはセッション録画でもレガシー録画でも共通で `recording_metadata` に設定する
   - report ファイルに `recording_metadata` または `metadata` のキーが存在しない場合にはウェブフックにも `recording_metadata` を含めない
   - @tnamao
+
+### misc
+
+- [CHANGE] GitHub Actions の ubuntu-latest を ubuntu-24.04 に変更
+  - @voluntas
 - [UPDATE] CI の staticcheck を 2024.1.1 にアップデート
   - @voluntas
 - [UPDATE] go 1.23.2 にアップデート

s3.go

@@ -181,9 +181,10 @@ func uploadWebMFileWithRateLimit(ctx context.Context, osConfig *s3.S3CompatibleO
 		Str("dst", dst).
 		Msg("WEB-UPLOAD-START")
 
-	// 制限時にはマルチパートアップロードを行わない
+	// 使用帯域の制限時は、巨大なサイズのファイルのアップロードする時に使用される multipart アップロードで
+	// 並列アップロードは行わずに 1 thread で処理されるようにオプションを設定する
 	n, err := s3Client.PutObject(ctx, osConfig.BucketName, dst, fileReader, fileSize,
-		minio.PutObjectOptions{ContentType: "application/octet-stream", DisableMultipart: true})
+		minio.PutObjectOptions{ContentType: "application/octet-stream", NumThreads: 1})
 	if err != nil {
 		return "", err
 	}