Do not set endpoint if it is an empty string

Also do not pass along endpoint as an override if we set it via the builder. Fixes #229.
gaul · Jul 9, 2017 · 17294be · 17294be
1 parent 23705a8
commit 17294be
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/main/java/org/gaul/s3proxy/Main.java b/src/main/java/org/gaul/s3proxy/Main.java
@@ -272,6 +272,7 @@ private static BlobStore createBlobStore(Properties properties)
         String credential = properties.getProperty(
                 Constants.PROPERTY_CREDENTIAL);
         String endpoint = properties.getProperty(Constants.PROPERTY_ENDPOINT);
+        properties.remove(Constants.PROPERTY_ENDPOINT);
         String region = properties.getProperty(
                 LocationConstants.PROPERTY_REGION);
 
@@ -306,7 +307,7 @@ private static BlobStore createBlobStore(Properties properties)
                 .credentials(identity, credential)
                 .modules(ImmutableList.<Module>of(new SLF4JLoggingModule()))
                 .overrides(properties);
-        if (endpoint != null) {
+        if (!Strings.isNullOrEmpty(endpoint)) {
             builder = builder.endpoint(endpoint);
         }