diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 3f22f83..9e5100a 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -64,7 +64,7 @@ jobs: - if: ${{ matrix.run-coveralls }} name: Coveralls - uses: coverallsapp/github-action@v2 + uses: coverallsapp/github-action@8d6379e14d29928660c4ba802d8e85393440b329 with: # *base-path* is prepended to all paths in order to correctly reference source files on coveralls.io base-path: src/main/java diff --git a/src/main/java/me/desair/tus/server/rufh/validation/RufhAppendValidator.java b/src/main/java/me/desair/tus/server/rufh/validation/RufhAppendValidator.java index ea92315..194f40f 100644 --- a/src/main/java/me/desair/tus/server/rufh/validation/RufhAppendValidator.java +++ b/src/main/java/me/desair/tus/server/rufh/validation/RufhAppendValidator.java @@ -53,6 +53,10 @@ public void validate( String ownerKey) throws TusException, IOException { + if (uploadStorageService == null) { + return; + } + String requestUri = request.getRequestURI(); boolean isCreationEndpoint = Utils.isCreationEndpoint(request, uploadStorageService); UploadInfo uploadInfo = uploadStorageService.getUploadInfo(requestUri, ownerKey); diff --git a/src/test/java/me/desair/tus/server/rufh/validation/RufhAppendValidatorTest.java b/src/test/java/me/desair/tus/server/rufh/validation/RufhAppendValidatorTest.java index 1513508..e1f9482 100644 --- a/src/test/java/me/desair/tus/server/rufh/validation/RufhAppendValidatorTest.java +++ b/src/test/java/me/desair/tus/server/rufh/validation/RufhAppendValidatorTest.java @@ -113,6 +113,12 @@ public void testValidateUploadInfoNull() throws Exception { validator.validate(HttpMethod.PATCH, request, storageService, "owner"); } + @Test + public void testValidateStorageServiceNull() throws Exception { + // Should return immediately without exception + validator.validate(HttpMethod.PATCH, request, null, "owner"); + } + @Test(expected = TusException.class) public void testValidateMismatchingUploadLength() throws Exception { request.setRequestURI("/files/exists");