Instructions for running "real FTP access" test are unclear

[ssteiner]

_test_real_ftp.py Test for real FTP access.

This test writes some files and directories on the local client and the remote server. Thus, you may want to skip this test by pressing [Ctrl-C].

If the test should run, enter the login data for the remote server.

This is not clear -- enter the login data how and where?

[schwa]

Yes, that's not so great. Please look at function get_login_data near the top of _test_real_ftp.py and adapt it.

A longer-term approach might be to take the credentials from a file which is only readable by the user running the tests. Using environment variables for userid and password is dangerous because the environment of a user can be inspected by other users, so they're able to see the password.

At least the text output by _test_real_ftp.py should be fixed to tell to change get_login_data.

[ssteiner]

There's always .netrc, we could just tell them to add an entry, and prompt for the server name to connect to.

[schwa]

I did a workaround in changeset [ca06daa965b8]. I agree that there should be a cleaner solution.

[schwa]

I hope the workaround is good enough, so I close this bug for now. If someone strongly disagrees they may reopen the ticket. ;-)