~sschwarzer/ftputil#48: 
Instructions for running "real FTP access" test are unclear

_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?

Status
RESOLVED FIXED
Submitter
ssteiner (unverified)
Assigned to
No-one
Submitted
13 years ago
Updated
13 years ago
Labels
bug library

schwa (unverified) 13 years ago · edit

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 (unverified) 13 years ago · edit

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

schwa (unverified) 13 years ago · edit

I did a workaround in changeset fe9d4f6941eb4644e426478efe2c76834c5656d1. I agree that there should be a cleaner solution.

schwa (unverified) 13 years ago · edit

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

Register here or Log in to comment, or comment via email.