Re: [PATCH/RFC] push: anonymize URL in error output

2017-08-24 Thread Ivan Vyshnevskyi
On 23/08/17 18:58, Jeff King wrote: > On Wed, Aug 23, 2017 at 12:49:29PM +0300, Ivan Vyshnevskyi wrote: > >> Commits 47abd85 (fetch: Strip usernames from url's before storing them, >> 2009-04-17) and later 882d49c (push: anonymize URL in status output, >> 2016-07-14)

Re: [PATCH/RFC] push: anonymize URL in error output

2017-08-24 Thread Ivan Vyshnevskyi
On 23/08/17 13:57, Lars Schneider wrote: > >> On 23 Aug 2017, at 11:49, Ivan Vyshnevskyi wrote: >> >> Commits 47abd85 (fetch: Strip usernames from url's before storing them, >> 2009-04-17) and later 882d49c (push: anonymize URL in status output, >> 201

[PATCH/RFC] push: anonymize URL in error output

2017-08-23 Thread Ivan Vyshnevskyi
rk failure will cause a push to fail, leaving a non-anonymized URL in the build log. To prevent that, reuse the same anonymizing function to scrub credentials from URL in the push error output. Signed-off-by: Ivan Vyshnevskyi --- This is my first attempt to propose a patch, sorry if I did somethi