Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Git-SVN: GIT_ASKPASS is buggy #43

Closed
dkp85 opened this issue Nov 23, 2012 · 4 comments
Closed

Git-SVN: GIT_ASKPASS is buggy #43

dkp85 opened this issue Nov 23, 2012 · 4 comments

Comments

@dkp85
Copy link

dkp85 commented Nov 23, 2012

When you use GIT_ASKPASS to enter passwords, it never works because function _read_password from Prompt.pm is buggy. Indeed, this line is supposed to remove \r\n:

$password =~ s/[\012\015]//; # \n\r

But a 'g' is missing so \r\n is never removed. The correct line is

$password =~ s/[\012\015]//g; # \n\r

The issue can be reproduced using git-gui--askpass.

@csware
Copy link

csware commented Jan 14, 2013

My patch is going to be applied in git 1.8.2.

@sschuberth
Copy link

@csware In fact your ss/svn-prompt was already merged for the just released Git 1.8.1.1 maintenance release, see http://thread.gmane.org/gmane.comp.version-control.git/213522.

@sschuberth
Copy link

@patthoyts recently merged upstream Git 1.8.1.1 into our master branch, so the next Git for Windows release will include @csware's fix.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants