git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: http getpass function in msysgit

From
Frank Li <lznuaa@gmail.com>
Date
Feb 4, 2010, 01:56 UTC
Message-ID
<1976ea661002031756j2c2699b0hf7deb8febdce841b@mail.gmail.com>
In-Reply-To
<7vaavp3i2n.fsf@alter.siamese.dyndns.org>
Show 22 quoted lines
>
>  - If GIT_ASKPASS is not set:
>
>   - If SSH_ASKPASS is present, then use that from getpass() for any and
>     all places that would want to get "password" like things;
>
>   - Otherwise consult the terminal as before;
>
>  - If GIT_ASKPASS is set:
>
>   - If SSH_ASKPASS is not set, then export the value of GIT_ASKPASS as
>     such as well, so that whenever we spawn "ssh", the same GIT_ASKPASS
>     program will be used as a fallback.
>
> That way, if the user already has set up SSH_ASKPASS, we will use the same
> familiar dialog without forcing the user do anything extra.  If the user
> only sets GIT_ASKPASS without doing SSH_ASKPASS, we would also use it to
> drive the ssh session.  In either case, the user doesn't need to worry
> about multiple configuration or dialog interface.
>
>> It is nice to use one dialog for all cases. git-svn also have the same problem.
>

Can we direct use SSH_ASKPASS for all getpass and don't use GIT_ASKPASS at all? The logic will be come simple.

Previous: Junio C HamanoNext: Junio C Hamano
Message 5 of 6 in “http getpass function in msysgit”
  1. Frank LiFeb 3, 2010
  2. Junio C HamanoFeb 3, 2010
  3. Frank LiFeb 3, 2010
  4. Junio C HamanoFeb 4, 2010
  5. Frank LiFeb 4, 2010
  6. Junio C HamanoFeb 4, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.