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

Git-SVN: GIT_ASKPASS is buggy #43

Closed

Description

@dkp85

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.

Activity

  1. csware commented on Jan 14, 2013

    @csware

    My patch is going to be applied in git 1.8.2.

  2. sschuberth commented on Jan 15, 2013

    @sschuberth

    @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.

  3. sschuberth commented on Jan 17, 2013

    @sschuberth

    @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 join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions