You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@archiva.apache.org by ma...@apache.org on 2019/05/25 08:51:32 UTC

[archiva] 03/07: Fixing exception handling for filesystem storage initialization

This is an automated email from the ASF dual-hosted git repository.

martin_s pushed a commit to branch feature/storage_refactoring
in repository https://gitbox.apache.org/repos/asf/archiva.git

commit aabd20f66839032f335f19b29dec0098fbffb178
Author: Martin Stockhammer <ma...@apache.org>
AuthorDate: Sat May 25 10:30:34 2019 +0200

    Fixing exception handling for filesystem storage initialization
---
 .../content/maven2/ManagedDefaultRepositoryContent.java    | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/content/maven2/ManagedDefaultRepositoryContent.java b/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/content/maven2/ManagedDefaultRepositoryContent.java
index 92c9b60..c34c01d 100644
--- a/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/content/maven2/ManagedDefaultRepositoryContent.java
+++ b/archiva-modules/archiva-maven/archiva-maven-repository/src/main/java/org/apache/archiva/repository/content/maven2/ManagedDefaultRepositoryContent.java
@@ -83,7 +83,12 @@ public class ManagedDefaultRepositoryContent
         super(Collections.singletonList( new DefaultArtifactMappingProvider() ));
         setFileTypes( fileTypes );
         setRepository( repository );
-        storage = new FilesystemStorage(getRepoDir(), lockManager);
+        try {
+            storage = new FilesystemStorage(getRepoDir(), lockManager);
+        } catch (IOException e) {
+            log.error("Could not initialize the filesystem storage to repository: {}", getRepoDir());
+            throw new RuntimeException("Fatal error. Could not initialize the filesystem storage for "+getRepoDir());
+        }
     }
 
     public ManagedDefaultRepositoryContent( ManagedRepository repository, List<? extends ArtifactMappingProvider> artifactMappingProviders, FileTypes fileTypes, FileLockManager lockManager )
@@ -91,7 +96,12 @@ public class ManagedDefaultRepositoryContent
         super(artifactMappingProviders==null ? Collections.singletonList( new DefaultArtifactMappingProvider() ) : artifactMappingProviders);
         setFileTypes( fileTypes );
         setRepository( repository );
-        storage = new FilesystemStorage(getRepoDir(), lockManager);
+        try {
+            storage = new FilesystemStorage(getRepoDir(), lockManager);
+        } catch (IOException e) {
+            log.error("Could not initialize the filesystem storage to repository: {}", getRepoDir());
+            throw new RuntimeException("Fatal error. Could not initialize the filesystem storage for "+getRepoDir());
+        }
     }
 
     private Path getRepoDir() {