commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From joe...@apache.org
Subject svn commit: r1448645 - in /commons/proper/vfs/trunk: core/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileSystemConfigBuilder.java src/changes/changes.xml
Date Thu, 21 Feb 2013 14:11:58 GMT
Author: joehni
Date: Thu Feb 21 14:11:58 2013
New Revision: 1448645

URL: http://svn.apache.org/r1448645
Log:
ConfigBuilder does not consider system properties for the value of SoTimeout and Encoding.

Modified:
    commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileSystemConfigBuilder.java
    commons/proper/vfs/trunk/src/changes/changes.xml

Modified: commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileSystemConfigBuilder.java
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileSystemConfigBuilder.java?rev=1448645&r1=1448644&r2=1448645&view=diff
==============================================================================
--- commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileSystemConfigBuilder.java
(original)
+++ commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileSystemConfigBuilder.java
Thu Feb 21 14:11:58 2013
@@ -80,7 +80,7 @@ public class FtpFileSystemConfigBuilder 
      */
     public Integer getConnectTimeout(final FileSystemOptions opts)
     {
-        return (Integer) getParam(opts, CONNECT_TIMEOUT);
+        return getInteger(opts, CONNECT_TIMEOUT);
     }
 
     /**
@@ -90,7 +90,7 @@ public class FtpFileSystemConfigBuilder 
      * */
     public String getControlEncoding(final FileSystemOptions opts)
     {
-        return (String) getParam(opts, ENCODING);
+        return getString(opts, ENCODING);
     }
 
     /**
@@ -205,7 +205,7 @@ public class FtpFileSystemConfigBuilder 
      */
     public Integer getSoTimeout(final FileSystemOptions opts)
     {
-        return (Integer) getParam(opts, SO_TIMEOUT);
+        return getInteger(opts, SO_TIMEOUT);
     }
 
     /**

Modified: commons/proper/vfs/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/src/changes/changes.xml?rev=1448645&r1=1448644&r2=1448645&view=diff
==============================================================================
--- commons/proper/vfs/trunk/src/changes/changes.xml (original)
+++ commons/proper/vfs/trunk/src/changes/changes.xml Thu Feb 21 14:11:58 2013
@@ -26,6 +26,9 @@
 <!--       <action issue="VFS-443" dev="ggregory" type="update" due-to="nickallen">
-->
 <!--     	[Local] Need an easy way to convert from a FileObject to a File. -->
 <!--       </action> -->
+      <action issue="VFS-461" dev="joehni" type="fix">
+        [FTP/FTPS] ConfigBuilder does not consider system properties for the value of SoTimeout
and Encoding.
+      </action>
       <action issue="VFS-412" dev="joehni" type="add" due-to="Jose Juan Montiel">
         [FTPS] Add support for command to set the DataChannelProtectionLevel.
       </action>



Mime
View raw message