You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2019/03/06 14:19:41 UTC

[commons-vfs] branch master updated: Add support for customizing FTP transfer aborted status codes. Update remaining "Status Code" to "Reply Code" in APIs and constants to match FTP terminology.

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

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-vfs.git


The following commit(s) were added to refs/heads/master by this push:
     new 1da4fb6  Add support for customizing FTP transfer aborted status codes. Update remaining "Status Code" to "Reply Code" in APIs and constants to match FTP terminology.
1da4fb6 is described below

commit 1da4fb6b4c6221566e12adfa53f1ac01fae38531
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Wed Mar 6 09:19:39 2019 -0500

    Add support for customizing FTP transfer aborted status codes. Update
    remaining
    "Status Code" to "Reply Code" in APIs and constants to match FTP
    terminology.
---
 .../main/java/org/apache/commons/vfs2/provider/ftp/FtpFileObject.java | 4 ++--
 .../apache/commons/vfs2/provider/ftp/FtpFileSystemConfigBuilder.java  | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileObject.java b/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileObject.java
index 6a1e52f..8f0bac6 100644
--- a/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileObject.java
+++ b/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileObject.java
@@ -589,10 +589,10 @@ public class FtpFileObject extends AbstractFileObject<FtpFileSystem> {
         }
 
         private boolean isTransferAbortedOkReplyCode() throws IOException {
-            List<Integer> transferAbortedOkStatusCodes = FtpFileSystemConfigBuilder
+            List<Integer> transferAbortedOkReplyCodes = FtpFileSystemConfigBuilder
                 .getInstance()
                 .getTransferAbortedOkReplyCodes(getAbstractFileSystem().getFileSystemOptions());
-            return transferAbortedOkStatusCodes != null && transferAbortedOkStatusCodes.contains(client.getReplyCode());
+            return transferAbortedOkReplyCodes != null && transferAbortedOkReplyCodes.contains(client.getReplyCode());
         }
     }
 
diff --git a/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileSystemConfigBuilder.java b/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileSystemConfigBuilder.java
index 981e645..66f75d0 100644
--- a/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileSystemConfigBuilder.java
+++ b/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/ftp/FtpFileSystemConfigBuilder.java
@@ -80,7 +80,7 @@ public class FtpFileSystemConfigBuilder extends FileSystemConfigBuilder {
         return FtpFileSystem.class;
     }
 
-    public static List<Integer> getSaneTransferAbortedOkStatusCodes() {
+    public static List<Integer> getSaneTransferAbortedOkReplyCodes() {
         // See VFS-674, its accompanying PR and https://github.com/apache/commons-vfs/pull/51 as to why 426 and 550 are here
         return new ArrayList<>(Arrays.asList(FTPReply.TRANSFER_ABORTED, FTPReply.FILE_UNAVAILABLE));
     }