Re: [PATCH] svn_load_dirs.pl: do not print password to screen (v2)

geoffrey.alary_at_gmail.com wrote:
> This is the second version of the patch, including the suggestion from
> Daniel.

Thank you! Looks good to me. I did a basic 'smoke test', just as far as
checking that during its first few 'svn log' commands it does indeed
hide the password in its output.

Committed in http://svn.apache.org/r1863262 .

- Julian

> Log message:
> [[[
> Do not print password to screen in svn_load_dirs.pl.
>
> * contrib/client-side/svn_load_dirs/svn_load_dirs.pl.in
> (sanitize_pwd): New function.
> (safe_read_from_pipe, read_from_process): Update the sites printing
> the command line to screen to use sanitize_pwd.
> (usage): Warn that other local OS users may be able to see the
> password passed on the command-line.
>
> Fix indentation; that is, replace the 2 tab occurrences by 8 spaces.
> ]]]
Received on 2019-07-18 11:47:19 CEST