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

Re: What's cooking in git.git (Jul 2016, #06; Tue, 19)

From
Lars Schneider <larsxschneider@gmail.com>
Date
Jul 20, 2016, 08:20 UTC
Message-ID
<CBEE46F3-5497-43D6-BA5A-217C7AD55B48@gmail.com>
In-Reply-To
<xmqqk2ghdysg.fsf@gitster.mtv.corp.google.com>
On 20 Jul 2016, at 00:54, Junio C Hamano <gitster@pobox.com> wrote:
Show 13 quoted lines
> 
> [New Topics]
> 
> ...
> 
> * jk/push-scrub-url (2016-07-14) 1 commit
>  (merged to 'next' on 2016-07-19 at 6ada3f1)
> + push: anonymize URL in status output
> 
> "git fetch http://user:pass@host/repo..." scrubbed the userinfo
> part, but "git push" didn't.
> 
> Will merge to 'master'.

t5541-http-push-smart.sh "push status output scrubs password" fails on next using Travis CI OS X: https://travis-ci.org/git/git/jobs/145960712 https://api.travis-ci.org/jobs/145960712/log.txt?deansi=true (non JS)

I think the test either fails because of OS X or because of the used Apache version (Travis CI Linux uses 2.2.22 and OS X uses 2.2.26). I haven't done any further investigation.

According to my Travis CI Git bisect script the failure starts to appear with the commit that introduced the test in 882d49c "push: anonymize URL in status output": https://travis-ci.org/larsxschneider/git/jobs/146027836

- Lars
Previous: Junio C HamanoNext: Jeff King
Message 2 of 6 in “What's cooking in git.git (Jul 2016, #06; Tue, 19)”
  1. Junio C HamanoJul 19, 2016
  2. Lars SchneiderJul 20, 2016
  3. t5541: fix url scrubbing test when GPG is not setJeff King, Jul 20, 2016
  4. Lars SchneiderJul 21, 2016
  5. Pranit BauvaJul 20, 2016
  6. Junio C HamanoJul 22, 2016

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.