[issue557704] netrc module can't handle all passwords

2016-11-10 Thread Cristian Măgherușan-Stanciu
Cristian Măgherușan-Stanciu added the comment: Is there anything blocking this from being really fixed? It's still broken on 3.5. The patch added two years ago works well for quoted passwords, I think that's good enough, anyway having some support is much better than the current out of the bo

[issue557704] netrc module can't handle all passwords

2016-01-18 Thread R. David Murray
R. David Murray added the comment: If it is a matter of following "the normal rules" about quoting in a place where we currently don't do that, I think it would be sensible to add it, but IMO it should be the full set of "normal rules". Presumably shlex provides facilities to do that...I have

[issue557704] netrc module can't handle all passwords

2016-01-17 Thread Martin Dengler
Martin Dengler added the comment: Bram's patch for "special" characters is in, mine is the one that allows spaces in .netrc by enabling the parsing of a password field's value that's surrounded by lexer.quotes ( https://hg.python.org/cpython/file/2.7/Lib/shlex.py#l45 ). This is not the same as

[issue557704] netrc module can't handle all passwords

2016-01-17 Thread Guido van Rossum
Guido van Rossum added the comment: (Hi Bram! :-) So does your patch also accept escaped spaces? I wonder if one of the problems here may not be that the syntax required to escape special characters isn't specified? That might be acceptable in 2003, not so much in 2016. -- __

[issue557704] netrc module can't handle all passwords

2016-01-17 Thread Martin Dengler
Martin Dengler added the comment: Anecdotal ( http://stackoverflow.com/a/12675195/2747741 ) evidence suggests other programs do indeed accept spaces, and a cursory browsing of the Perl source code ( http://perl5.git.perl.org/perl.git/blob/HEAD:/cpan/libnet/lib/Net/Netrc.pm#l120 ) indicates Pe

[issue557704] netrc module can't handle all passwords

2015-12-18 Thread R. David Murray
R. David Murray added the comment: To clarify: other FTP programs handling passwords with spaces *in the .netrc file*. -- ___ Python tracker ___ ___

[issue557704] netrc module can't handle all passwords

2015-12-18 Thread R. David Murray
R. David Murray added the comment: This issue was closed because other FTP programs also did not handle passwords with spaces. If this has subsequently changed (passwords with spaces are now widely accepted by other FTP programs) then the issue could be reopened. -- nosy: +r.david.mur

[issue557704] netrc module can't handle all passwords

2015-11-16 Thread Cristian Măgherușan-Stanciu
Cristian Măgherușan-Stanciu added the comment: Why is this issue fixed? I still see this problem on 2.7 and 3.4.3. Can someone please reopen it? mdengler's patch seems to work fine on my machine on both 2.7 and 3.4.3. -- nosy: +Cristian Măgherușan-Stanciu _

[issue557704] netrc module can't handle all passwords

2014-10-09 Thread Martin Dengler
Martin Dengler added the comment: Sorry for the whitespace-unaware diff. The attached patch is the real one, with the obvious extra level of indentation around the critical "password = lexer.get_token()" line. -- keywords: +patch Added file: http://bugs.python.org/file36851/python-ne

[issue557704] netrc module can't handle all passwords

2014-10-09 Thread Martin Dengler
Martin Dengler added the comment: I know this is ancient, but the below patch handles spaces in passwords in 2.7.8 and 3.4 for me. If this is worth making into a new bug / proper patch I'm happy to do it. $ diff -uw /c/Python278/Lib/netrc.py{-orig,} --- /c/Python278/Lib/netrc.py-orig 201