_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?
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 changeget_login_data
.
There's always .netrc, we could just tell them to add an entry, and prompt for the server name to connect to.
I did a workaround in changeset fe9d4f6941eb4644e426478efe2c76834c5656d1. I agree that there should be a cleaner solution.
I hope the workaround is good enough, so I close this bug for now. If someone strongly disagrees they may reopen the ticket. ;-)