Ticket #1796 (closed defect: fixed)

Opened 10 years ago

Last modified 9 years ago

FTPFS does not work in active mode

Reported by: bszente Owned by: slavazanko
Priority: major Milestone: 4.7.0
Component: mc-vfs Version: 4.7.0-pre4
Keywords: Cc: balint@…
Blocked By: Blocking:
Branch state: Votes for changeset: committed-master

Description

After upgrading from 4.7.0_pre1 to 4.7.0_pre4 the FTP does not work in active mode anymore.

Connecting to a server in active mode gives the following error:

Cannot chdir to /#ftp:<server>

It might be possible that the bug was introduced in 4.7.0_pre2 with the IPv6 protocol support.

Attachments

ftp-4.7.0_pre1.log (647 bytes) - added by bszente 10 years ago.
FTP log file for working 4.7.0_pre1
ftp-4.7.0_pre4.log (212 bytes) - added by bszente 10 years ago.
FTP log file for non working 4.7.0_pre4

Change History

Changed 10 years ago by bszente

FTP log file for working 4.7.0_pre1

Changed 10 years ago by bszente

FTP log file for non working 4.7.0_pre4

comment:1 Changed 10 years ago by bszente

  • Cc balint@… added

It seems that the PORT command is not issued in active mode.

comment:2 Changed 10 years ago by slavazanko

  • Owner set to slavazanko
  • Status changed from new to accepted

comment:3 Changed 10 years ago by slavazanko

  • severity changed from no branch to on review

Created branch 1796_FTPFS_fail_active_mode

Initial changeset:852197c99d53973b6308fcac55042b40d6ba6077

Review, please.

comment:4 Changed 10 years ago by bszente

The fix is working. Thanks.

comment:5 Changed 10 years ago by andrew_b

  • Votes for changeset set to andrew_b

comment:6 Changed 10 years ago by angel_il

  • Votes for changeset changed from andrew_b to andrew_b angel_il
  • severity changed from on review to approved

comment:7 Changed 10 years ago by slavazanko

  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b angel_il to commited-master
  • Resolution set to fixed
  • severity changed from approved to merged

comment:8 Changed 10 years ago by slavazanko

  • Status changed from testing to closed

comment:9 Changed 10 years ago by slavazanko

  • Status changed from closed to reopened
  • Votes for changeset commited-master deleted
  • Resolution fixed deleted
  • severity changed from merged to on review

someplace treat, elsewhere broke...

No good works with passive ftp-mode.

Created branch 1796_fix_passive

Initial changeset:f3e2d16ab034364962e46d9e1edaf130f5ec595b

Review, please.

comment:10 Changed 10 years ago by slavazanko

  • Status changed from reopened to accepted

comment:11 Changed 10 years ago by zaytsev

rezult -> result

comment:12 Changed 10 years ago by metux

rebased and fixed typo in branch:1796_fix_passive_METUX.
(running it now through testfarm)

comment:13 Changed 10 years ago by andrew_b

Why you created new branch and didn't fix typo in existing one?

comment:14 Changed 10 years ago by slavazanko

As I see, branch '1796_fix_passive_METUX' contain just rebase * my one patch (originally from 1796_fix_passive).

Please:

  • no need to often rebase
  • no need to spawn too much branches.

Branch 1796_fix_passive_METUX now deleted.

comment:15 Changed 10 years ago by slavazanko

Review again, pleae.

comment:16 Changed 10 years ago by andrew_b

  • Votes for changeset set to andrew_b

comment:17 Changed 10 years ago by angel_il

  • Votes for changeset changed from andrew_b to andrew_b angel_il
  • severity changed from on review to approved

comment:18 Changed 10 years ago by slavazanko

  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b angel_il to commited-master
  • Resolution set to fixed
  • severity changed from approved to merged

comment:19 Changed 10 years ago by slavazanko

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.