[patch] Accept passwords from STDIN

PhilosChen / appdailysales

Automatically exported from code.google.com/p/appdailysales

0 stars 0 forks source link

[patch] Accept passwords from STDIN #7

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

Since passing passwords via command line arguments is a considerable 
security risk (imagine two people logged into a mac, you switched to your 
user but the other user still has a ps-scraping utility running, he can 
easily steal your iTc password) I've modified the script to be able to 
read the password from stdin.

Original issue reported on code.google.com by lhunath on 1 Aug 2009 at 6:34

Attachments:

password-stdin.patch

GoogleCodeExporter commented 8 years ago

Thanks! I'll add the patched version to the trunk today.

Question.  Why sys.stdin.read() instead of sys.stdin.readline()?

Original comment by ki...@thecave.com on 3 Aug 2009 at 3:09

GoogleCodeExporter commented 8 years ago

The patch has been applied and is available as version 1.9. Thanks for the 
contribution.

Original comment by ki...@thecave.com on 3 Aug 2009 at 3:25

Changed state: Fixed

GoogleCodeExporter commented 8 years ago

Mostly just because there's no point introducing a limitation on passwords 
having to 
be only one line long.

Original comment by lhunath on 4 Aug 2009 at 8:30

GoogleCodeExporter commented 8 years ago

Good point. Thanks.

Original comment by ki...@thecave.com on 4 Aug 2009 at 10:27