commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rory Winston (JIRA)" <j...@apache.org>
Subject [jira] Closed: (NET-54) [net] FTPReply.isPositivePreliminary(int) returns false when reply = 220 (Server ready) Causes FTPClient.retrieveFile(String, OutputStream) to fail
Date Wed, 11 Apr 2007 09:25:32 GMT

     [ https://issues.apache.org/jira/browse/NET-54?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Rory Winston closed NET-54.
---------------------------

    Resolution: Invalid

The code shown is correct. 220 is not a positive preliminary value, rather it is a positive
completion value. Hence isPositivePreliminary() should return false.

> [net] FTPReply.isPositivePreliminary(int) returns false when reply = 220 (Server ready)
Causes FTPClient.retrieveFile(String, OutputStream) to fail
> ---------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: NET-54
>                 URL: https://issues.apache.org/jira/browse/NET-54
>             Project: Commons Net
>          Issue Type: Bug
>    Affects Versions: 1.4
>         Environment: Operating System: other
> Platform: PC
>            Reporter: Jeff Barnes
>
> public static boolean isPositivePreliminary(int reply)
>     {
>         return (reply >= 100 && reply < 200);
>     }
> should be changed to
>     public static boolean isPositivePreliminary(int reply)
>     {
>         return (reply >= 100 && reply < 220); //or is it some other magic

> number?
>     }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message