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

Re: git_inetd_server: run git-http-backend using inetd

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Jul 18, 2014, 02:22 UTC
Message-ID
<20140718022208.GQ12427@google.com>
In-Reply-To
<8E3CD89B-2E47-4CA8-8F3D-598A9BE2AD9F@gmail.com>
Kyle J. McKay wrote:
> On Jul 17, 2014, at 15:10, Jonathan Nieder wrote:
Show 5 quoted lines
>> It was added to respond to a feature request within Debian but it is
>> intended to eventually go upstream.  I'm glad you found this issue
>> before that could happen. :)
>
> Is there some reason the patch is not opt-in at runtime?

Yes, it's a feature for server admins to control where clients connect, to control where the load goes. If the client has to enable it explicitly, that defeats the purpose.

I'd much rather fix it than turn it off completely.
[...]
Show 9 quoted lines
>> Should git automatically disable the SRV lookups when it sees one of
>> the six domains named in RFC6762, or is there some system library call
>> that can use mDNS when appropriate automatically (or get partway there
>> without having to hard-code the domains)?
>
> Sadly I think mDNS support is relegated to an add-on package on
> Linux.  And Avahi [1] is the package that generally provides it
> there.  The recommended interface for C is the avahi-client API (see
> [2]).  However, that is Avahi-only.
[...]
Show 8 quoted lines
> Even if the choice is to just disable SRV lookups for mDNS hosts at
> a minimum the code will have to determine whether or not the given
> host name is a mDNS name and it can't reliably do that for a host
> name without any dots in it without at least looking at the default
> domain name and possibly the search domain(s) as well.
>
> I think it would be much simpler just to make this opt-in via a
> config option
Thanks for these details.  I'll file a bug and mull it over some more.

RFC 6762 makes it clear that what the package is currently doing is wrong. Given that Debian's libc knows nothing about mdns on its own, I think I'll need to parse resolv.conf (that's what libc-ares does).

Jonathan
Previous: Kyle J. McKayNext: Kyle J. McKay
Message 4 of 13 in “git_inetd_server: run git-http-backend using inetd”
  1. Kyle J. McKayJul 17, 2014
  2. Jonathan NiederJul 17, 2014
  3. Kyle J. McKayJul 17, 2014
  4. Jonathan NiederJul 18, 2014
  5. Kyle J. McKayJul 18, 2014
  6. Jonathan NiederJul 18, 2014
  7. Kyle J. McKayJul 19, 2014
  8. Jonathan NiederJul 19, 2014
  9. Kyle J. McKayJul 19, 2014
  10. Torsten BögershausenJul 19, 2014
  11. Jonathan NiederJul 19, 2014
  12. Torsten BögershausenJul 20, 2014
  13. Torsten BögershausenJul 20, 2014

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.