geoffrey.al...@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.
]]]

Reply via email to