From 3d04ead71701a37c0ac29945f301930a6f412855 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=A0t=C4=9Bp=C3=A1n=20Brychta?= Date: Fri, 4 Oct 2024 10:49:35 +0100 Subject: [PATCH 1/3] Release action changes --- .github/workflows/release.yml | 2 ++ .github/workflows/report-evictions.yml | 1 + .github/workflows/run-tests.yml | 1 + 3 files changed, 4 insertions(+) diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index be5d9d10..39018781 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -92,10 +92,12 @@ jobs: runs-on: ubuntu-latest needs: create-release strategy: + fail-fast: false matrix: service: - fixtures - http + - http_typesafe - json - typesafe_app - monitoring diff --git a/.github/workflows/report-evictions.yml b/.github/workflows/report-evictions.yml index d12518db..aad3ca3a 100644 --- a/.github/workflows/report-evictions.yml +++ b/.github/workflows/report-evictions.yml @@ -17,6 +17,7 @@ jobs: service: - fixtures - http + - http_typesafe - json - typesafe_app - monitoring diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml index d64259a6..4e908ebf 100644 --- a/.github/workflows/run-tests.yml +++ b/.github/workflows/run-tests.yml @@ -15,6 +15,7 @@ jobs: service: - fixtures - http + - http_typesafe - json - typesafe_app - monitoring From 9750cce95856351890a46c9d9d5295462e53f233 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=A0t=C4=9Bp=C3=A1n=20Brychta?= Date: Mon, 7 Oct 2024 08:45:41 +0100 Subject: [PATCH 2/3] S3StreamWritable bug fix --- .../main/scala/weco/storage/store/s3/S3StreamWritable.scala | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/storage/src/main/scala/weco/storage/store/s3/S3StreamWritable.scala b/storage/src/main/scala/weco/storage/store/s3/S3StreamWritable.scala index 4df5a876..31552d39 100644 --- a/storage/src/main/scala/weco/storage/store/s3/S3StreamWritable.scala +++ b/storage/src/main/scala/weco/storage/store/s3/S3StreamWritable.scala @@ -41,7 +41,7 @@ trait S3StreamWritable if (inputStream.length > 0) { val bytes: Array[Byte] = new Array[Byte](inputStream.length.toInt) val bytesRead = - inputStream.read(bytes, 0, inputStream.length.toInt) + inputStream.readNBytes(bytes, 0, inputStream.length.toInt) if (bytesRead < inputStream.length) { throw new RuntimeException( @@ -93,7 +93,7 @@ trait S3StreamWritable val partLength = (end - start).toInt val bytes: Array[Byte] = new Array[Byte](partLength) - val bytesRead = inputStream.read(bytes, 0, partLength) + val bytesRead = inputStream.readNBytes(bytes, 0, partLength) if (bytesRead < partLength) { throw new RuntimeException( From 197d6a21e45905f752781e25d7d787bf1682755a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=A0t=C4=9Bp=C3=A1n=20Brychta?= Date: Mon, 7 Oct 2024 11:28:38 +0100 Subject: [PATCH 3/3] Create RELEASE.md --- RELEASE.md | 3 +++ 1 file changed, 3 insertions(+) create mode 100644 RELEASE.md diff --git a/RELEASE.md b/RELEASE.md new file mode 100644 index 00000000..208ffca2 --- /dev/null +++ b/RELEASE.md @@ -0,0 +1,3 @@ +RELEASE_TYPE: patch + +S3StreamWritable bug fix (replace `read` by `readNBytes`).