You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@archiva.apache.org by ol...@apache.org on 2016/09/26 22:40:43 UTC

archiva git commit: cosmetic :-)

Repository: archiva
Updated Branches:
  refs/heads/master 85be2b77a -> 229276f8d


cosmetic :-)

Signed-off-by: olivier lamy <ol...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/archiva/repo
Commit: http://git-wip-us.apache.org/repos/asf/archiva/commit/229276f8
Tree: http://git-wip-us.apache.org/repos/asf/archiva/tree/229276f8
Diff: http://git-wip-us.apache.org/repos/asf/archiva/diff/229276f8

Branch: refs/heads/master
Commit: 229276f8d933089dcfec615189ac31259aa45697
Parents: 85be2b7
Author: olivier lamy <ol...@apache.org>
Authored: Tue Sep 27 08:40:37 2016 +1000
Committer: olivier lamy <ol...@apache.org>
Committed: Tue Sep 27 08:40:37 2016 +1000

----------------------------------------------------------------------
 .../proxy/DefaultRepositoryProxyConnectors.java   | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/archiva/blob/229276f8/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/archiva/proxy/DefaultRepositoryProxyConnectors.java
----------------------------------------------------------------------
diff --git a/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/archiva/proxy/DefaultRepositoryProxyConnectors.java b/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/archiva/proxy/DefaultRepositoryProxyConnectors.java
index 864d806..5d60874 100644
--- a/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/archiva/proxy/DefaultRepositoryProxyConnectors.java
+++ b/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/archiva/proxy/DefaultRepositoryProxyConnectors.java
@@ -48,8 +48,8 @@ import org.apache.archiva.policies.urlcache.UrlFailureCache;
 import org.apache.archiva.proxy.common.WagonFactory;
 import org.apache.archiva.proxy.common.WagonFactoryException;
 import org.apache.archiva.proxy.common.WagonFactoryRequest;
-import org.apache.archiva.proxy.model.ProxyFetchResult;
 import org.apache.archiva.proxy.model.ProxyConnector;
+import org.apache.archiva.proxy.model.ProxyFetchResult;
 import org.apache.archiva.proxy.model.RepositoryProxyConnectors;
 import org.apache.archiva.redback.components.registry.Registry;
 import org.apache.archiva.redback.components.registry.RegistryListener;
@@ -90,7 +90,6 @@ import java.io.IOException;
 import java.nio.file.Files;
 import java.util.ArrayList;
 import java.util.Collections;
-import java.util.HashMap;
 import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
@@ -98,6 +97,7 @@ import java.util.Map.Entry;
 import java.util.Properties;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.ConcurrentMap;
+import java.util.concurrent.TimeUnit;
 
 /**
  * DefaultRepositoryProxyConnectors
@@ -1184,12 +1184,13 @@ public class DefaultRepositoryProxyConnectors
         }
 
         // Convert seconds to milliseconds
-        int timeoutInMilliseconds = remoteRepository.getRepository().getTimeout() * 1000;
+        long timeoutInMilliseconds = TimeUnit.MILLISECONDS.convert( remoteRepository.getRepository().getTimeout(), //
+                                                                    TimeUnit.SECONDS );
 
         // Set timeout  read and connect
         // FIXME olamy having 2 config values
-        wagon.setReadTimeout( timeoutInMilliseconds );
-        wagon.setTimeout( timeoutInMilliseconds );
+        wagon.setReadTimeout( (int) timeoutInMilliseconds );
+        wagon.setTimeout( (int)  timeoutInMilliseconds );
 
         try
         {
@@ -1263,9 +1264,10 @@ public class DefaultRepositoryProxyConnectors
     @Override
     public void afterConfigurationChange( Registry registry, String propertyName, Object propertyValue )
     {
-        if ( ConfigurationNames.isNetworkProxy( propertyName ) || ConfigurationNames.isManagedRepositories(
-            propertyName ) || ConfigurationNames.isRemoteRepositories( propertyName )
-            || ConfigurationNames.isProxyConnector( propertyName ) )
+        if ( ConfigurationNames.isNetworkProxy( propertyName ) //
+            || ConfigurationNames.isManagedRepositories( propertyName ) //
+            || ConfigurationNames.isRemoteRepositories( propertyName ) //
+            || ConfigurationNames.isProxyConnector( propertyName ) ) //
         {
             initConnectorsAndNetworkProxies();
         }