Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

FTPFS does not work in active mode #1796

Closed
mc-butler opened this issue Nov 5, 2009 · 22 comments
Closed

FTPFS does not work in active mode #1796

mc-butler opened this issue Nov 5, 2009 · 22 comments
Assignees
@slavaz
Labels
area: vfs Virtual File System support prio: medium Has the potential to affect progress ver: 4.7.0-pre4 Reproducible in version 4.7.0-pre4
Milestone
4.7.0

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/1796
Reporter bszente (balint@….ro)
Mentions balint@….ro

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.

Note

Original attachments:
@mc-butler
Copy link
Author

Changed by bszente (balint@….ro) on Nov 5, 2009 at 13:20 UTC

FTP log file for working 4.7.0_pre1

@mc-butler
Copy link
Author

Changed by bszente (balint@….ro) on Nov 5, 2009 at 13:20 UTC

FTP log file for non working 4.7.0_pre4

@mc-butler
Copy link
Author

Changed by bszente (balint@….ro) on Nov 5, 2009 at 13:22 UTC (comment 1)

  • Cc set to balint@….ro

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

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Nov 23, 2009 at 11:06 UTC (comment 2)

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

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Nov 24, 2009 at 9:32 UTC (comment 3)

  • Severity changed from no branch to on review

Created branch 1796_FTPFS_fail_active_mode

Initial [852197c99d53973b6308fcac55042b40d6ba6077]

Review, please.

@mc-butler
Copy link
Author

Changed by bszente (balint@….ro) on Nov 24, 2009 at 14:04 UTC (comment 4)

The fix is working. Thanks.

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 2, 2009 at 13:15 UTC (comment 5)

  • Votes set to andrew_b

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Dec 2, 2009 at 13:55 UTC (comment 6)

  • Severity changed from on review to approved
  • Votes changed from andrew_b to andrew_b angel_il

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 2, 2009 at 14:01 UTC (comment 7)

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

Merge [553bef6]

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 2, 2009 at 14:01 UTC (comment 8)

  • Status changed from testing to closed

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 10, 2009 at 20:25 UTC (comment 9)

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

someplace treat, elsewhere broke...

No good works with passive ftp-mode.

Created branch 1796_fix_passive

Initial [f3e2d16ab034364962e46d9e1edaf130f5ec595b]

Review, please.

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 10, 2009 at 20:26 UTC (comment 10)

  • Status changed from reopened to accepted

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Dec 11, 2009 at 21:06 UTC (comment 11)

rezult -> result

@mc-butler
Copy link
Author

Changed by metux (@metux) on Dec 13, 2009 at 4:38 UTC (comment 12)

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

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 13, 2009 at 7:52 UTC (comment 13)

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

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 15, 2009 at 10:31 UTC (comment 14)

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.

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 15, 2009 at 10:33 UTC (comment 15)

  • [3e57cb5a832fee9d7feae6e4feaf9ddd6213a0be]: FOR REBASE: fix typo (rezult -> result)

Review again, pleae.

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 15, 2009 at 11:01 UTC (comment 16)

  • Votes set to andrew_b

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Dec 15, 2009 at 11:15 UTC (comment 17)

  • Severity changed from on review to approved
  • Votes changed from andrew_b to andrew_b angel_il

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 15, 2009 at 11:18 UTC (comment 18)

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

merge [a0930b7]

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 15, 2009 at 11:19 UTC (comment 19)

  • Status changed from testing to closed

@mc-butler
Copy link
Author

Changed by slyfox (@trofi) on Dec 26, 2009 at 15:56 UTC (comment 20)

http://bugs.gentoo.org/show_bug.cgi?id=294421 (for history)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@slavaz slavaz

Labels
area: vfs Virtual File System support prio: medium Has the potential to affect progress ver: 4.7.0-pre4 Reproducible in version 4.7.0-pre4
Milestone
4.7.0
Development

No branches or pull requests
2 participants
@slavaz @mc-butler