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

Re: Make Gitweb behave like Apache mod_userdir

From
JJ.H. <warthog19@eaglescrag.net>
Date
Nov 18, 2009, 00:00 UTC
Message-ID
<4B03391A.90808@eaglescrag.net>
In-Reply-To
<7vskccaf4q.fsf@alter.siamese.dyndns.org>
Junio C Hamano wrote:
Show 21 quoted lines
> "J.H." <warthog19@eaglescrag.net> writes:
> 
>> For starters I think overriding the /~<user> (specifically the ~ here)
>> is going to be a bad idea no matter what you do and gives the wrong
>> impression about what / how the request is being responded to.  You
>> might want to try and pick a different delimiter or re-work the rule
>> so that you could have something like:
>>
>> 	http://git.kernel.org/<gitweb urls>
>> 	http://git.kernel.org/user/<gitweb urls>
> 
> I am not objecting but showing my ignorance, but why is it a bad idea to
> allow
> 
> 	http://git.kernel.org/~junio/git.git
> 
> to map to /home/junio/pubilic_git/git.git when
> 
> 	http://www.kernel.org/~junio/index.html
> 
> maps to /home/junio/public_html/index.html already?

I'm not objecting to the mapping, I'm objecting to the use of the ~ specifically, since that's the default for mod_userdir - it's more or less a race condition waiting to happen with respect to does mod_rewrite get to and process it first or does mod_userdir.

Case in point should:

assuming that www.example.com (to get away from the specifics of kernel.org) is running gitweb but you also wanted to allow users to show their ~/public_html directories then

	http://www.example.org/~junio/
could be ambiguous as to what should / is servicing the url.

Now in reality it should work because of how apache is setup, but I would argue it's a bad practice to overload the '~' designator like that for use in gitweb specifically.

If you have mod_userdir disabled / not available this isn't an issue, but making that assumption might be a bad idea.

- John 'Warthog9' Hawley
Previous: Junio C HamanoNext: Sylvain Rabot
Message 17 of 18 in “Make Gitweb behave like Apache mod_userdir”
  1. Sylvain RabotNov 15, 2009
  2. Junio C HamanoNov 15, 2009
  3. Jakub NarebskiNov 17, 2009
  4. Sylvain RabotNov 17, 2009
  5. Jakub NarebskiNov 17, 2009
  6. Sylvain RabotNov 17, 2009
  7. Junio C HamanoNov 17, 2009
  8. Sylvain RabotNov 17, 2009
  9. Junio C HamanoNov 17, 2009
  10. Sylvain RabotNov 17, 2009
  11. Junio C HamanoNov 17, 2009
  12. J.H.Nov 17, 2009
  13. J.H.Nov 17, 2009
  14. Junio C HamanoNov 17, 2009
  15. J.H.Nov 17, 2009
  16. Junio C HamanoNov 18, 2009
  17. J.H.Nov 18, 2009
  18. Sylvain RabotNov 18, 2009

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.