[Cyberduck-trac] [Cyberduck] #9603: Uncomfirmed Reads are Unbounded

Cyberduck trac at trac.cyberduck.io
Mon Jun 20 18:19:26 UTC 2016


#9603: Uncomfirmed Reads are Unbounded
-----------------------+-------------------------
 Reporter:  jbaranick  |         Owner:  dkocher
     Type:  defect     |        Status:  assigned
 Priority:  normal     |     Milestone:  5.1
Component:  sftp       |       Version:  5.0
 Severity:  normal     |    Resolution:
 Keywords:             |  Architecture:
 Platform:             |
-----------------------+-------------------------
Description changed by jbaranick:

Old description:

> If the file being downloaded is very large the number of unconfirmed
> reads is effectively unbounded.  For example, when the file is 1.3GB, the
> number of unconfirmed reads is set to 39K.  This results in the client
> submitting 39K calls to the server, updating the progress to look like
> the download is done, and then failing when some of the calls don't get
> responses in time.  As far as I can tell, there is no way for the server
> to push back against these requests.  As such, Cyberduck should limit the
> number of unconfirmed reads to something more reasonable, for example 1K.
>
> FYI, the command like SFTP client supports 64 unconfirmed requests by
> default:
>
> {{{
> ~ ❯❯❯ sftp -R
>  -- number of outstanding requests (default: 64) --
> }}}

New description:

 If the file being downloaded is very large the number of unconfirmed reads
 is effectively unbounded.  For example, when the file is 1.3GB, the number
 of unconfirmed reads is set to 39K.  This results in the client submitting
 39K calls to the server, updating the progress to look like the download
 is done, and then failing when some of the calls don't get responses in
 time.  As far as I can tell, there is no way for the server to push back
 against these requests.  As such, Cyberduck should limit the number of
 unconfirmed reads to something more reasonable, for example 1K.

 FYI, the command like SFTP client supports 64 unconfirmed requests by
 default:

 {{{
 ~ ❯❯❯ sftp -R
  -- number of outstanding requests (default: 64)
 }}}

--

--
Ticket URL: <https://trac.cyberduck.io/ticket/9603#comment:3>
Cyberduck <https://cyberduck.io>
Libre FTP, SFTP, WebDAV, S3 & OpenStack Swift browser for Mac and Windows


More information about the Cyberduck-trac mailing list