fails ftp directory listing
Reported by
pdk...@gmail.com,
Aug 3 2016
|
|||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.82 Safari/537.36 Example URL: ftp://ftp.graphicsmagick.org/pub/GraphicsMagick/1.3/ Steps to reproduce the problem: ^ What is the expected behavior? What went wrong? The directory listing is empty. Works fine in Firefox. Does it occur on multiple sites: N/A Is it a problem with a plugin? No Did this work before? N/A Does this work in other browsers? Yes Chrome version: 52.0.2743.82 Channel: stable OS Version: Ubuntu 14.04 Flash Version:
,
Dec 18 2016
Still in M56.
,
Mar 5 2017
Still in M57 - but different. The page keeps loading.
,
Apr 18 2017
Since ftp listings are implemented with HTML/CSS/JS underneath, one can open up developer tools and see: Failed to load resource: net::ERR_EMPTY_RESPONSE
,
Apr 18 2017
Looking in chrome://net-internals, at the very end of the FTP directory listing...
=1337 [st=1227] FTP_CONTROL_RESPONSE
--> lines = ["Transfer complete"]
--> status_code = 226
t=1337 [st=1227] FTP_COMMAND_SENT
--> command = "QUIT"
t=1418 [st=1308] FAILED
--> net_error = -324 (ERR_EMPTY_RESPONSE)
t=1419 [st=1309] -REQUEST_ALIVE
--> net_error = -324 (ERR_EMPTY_RESPONSE)
Maybe net/ftp/ftp_network_transaction.cc has the answers.
,
Apr 20 2017
We need to ignore empty quit responses for this one.
,
Apr 21 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e1eca6ba538bdcbd4fe8d78c9bb208ec1b818702 commit e1eca6ba538bdcbd4fe8d78c9bb208ec1b818702 Author: thestig <thestig@chromium.org> Date: Fri Apr 21 18:05:59 2017 Handle empty response to FTP QUIT message. Just pretend it is ok, rather than returning an error. BUG= 633841 Review-Url: https://codereview.chromium.org/2836503002 Cr-Commit-Position: refs/heads/master@{#466388} [modify] https://crrev.com/e1eca6ba538bdcbd4fe8d78c9bb208ec1b818702/net/ftp/ftp_network_transaction.cc [modify] https://crrev.com/e1eca6ba538bdcbd4fe8d78c9bb208ec1b818702/net/ftp/ftp_network_transaction.h [modify] https://crrev.com/e1eca6ba538bdcbd4fe8d78c9bb208ec1b818702/net/ftp/ftp_network_transaction_unittest.cc
,
Apr 21 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by mattm@chromium.org
, Aug 3 2016Status: Untriaged (was: Unconfirmed)