From: "Carlo Marcelo Arenas Belón" <email@example.com> To: Christian Couder <firstname.lastname@example.org> Cc: Junio C Hamano <email@example.com>, git <firstname.lastname@example.org>, Christian Couder <email@example.com>, Derrick Stolee <firstname.lastname@example.org>, Jacob Vosmaer <email@example.com> Subject: Re: [PATCH] http: add custom hostname to IP address resolves Date: Thu, 5 May 2022 04:16:08 -0700 [thread overview] Message-ID: <firstname.lastname@example.org> (raw) In-Reply-To: <CAP8UFD0V6scSD8LuUiRQ0BWmG2mFATVFNQ2DgwaWFp6bigBkZw@mail.gmail.com> On Thu, May 05, 2022 at 12:48:50PM +0200, Christian Couder wrote: > On Wed, May 4, 2022 at 4:34 PM Junio C Hamano <email@example.com> wrote: > > > > > > I am also interested in others' thoughts about this. If no one thinks > > > that a config option could be useful, I am Ok with making it a > > > "--resolve" command line option that can be passed to any Git command > > > similar to "-c <name>=<value>": > > > > > > git --resolve=... <command> [<args>] > > > > Absolutely not. > > > > "git [push|fetch|clone|ls-remote] --dns-pre-resolve=..." that is > > *NOT* git wide, but is only for transport commands might be a > > possibility, but even then, you'd need to invent a way to do the > > same for non cURL transports (we want to be able to pin the IP when > > going over SSH to a certain host, for the same reason) if we promote > > it to an officially supported command line option. > > Ok with renaming and implementing it only in transport commands. I > don't want, and I don't think it should be necessary, to invent a way > to do the same for non cURL transports though. I think it should be Ok > with the doc saying that the option has only been implemented for > HTTP(S) yet and will have no effect when other transports are used. I think it will be better if git aborts with an error if it is used for a transport that it doesn't support, instead of relying in the documentation, though. > If there is a simple way to do the same thing for ssh, then I might > take a look at it later. For "file" or bundle transports, I don't > think it makes sense, and the "git" transport is not used much in big > hosting services where this feature is likely to be used. This seems definitely useful also for ssh which is also used in big hosting services. Ironically, I think would be even more useful for the "git" transport specially because it doesn't have other protections to rely on that would help prevent spoofing (like TLS), which might be also why it is not that widely used anymore. Carlo
next prev parent reply other threads:[~2022-05-05 11:16 UTC|newest] Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-05-02 8:36 Christian Couder 2022-05-02 19:04 ` Junio C Hamano 2022-05-04 10:07 ` Christian Couder 2022-05-04 14:34 ` Junio C Hamano 2022-05-05 10:48 ` Christian Couder 2022-05-05 11:16 ` Carlo Marcelo Arenas Belón [this message] 2022-05-09 15:40 ` Christian Couder 2022-05-04 10:46 ` [PATCH v2] http: add custom hostname to IP address resolutions Christian Couder 2022-05-05 11:21 ` Carlo Marcelo Arenas Belón 2022-05-12 8:52 ` Christian Couder 2022-05-12 16:22 ` Junio C Hamano 2022-05-12 18:57 ` Christian Couder 2022-05-09 15:38 ` [PATCH v3] " Christian Couder 2022-05-10 18:20 ` Carlo Arenas 2022-05-12 8:29 ` Christian Couder 2022-05-12 11:55 ` Carlo Arenas 2022-05-12 13:01 ` Patrick Steinhardt 2022-05-12 13:56 ` Carlo Arenas 2022-05-12 15:58 ` Junio C Hamano 2022-05-16 8:38 ` [PATCH v4] " Christian Couder
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style List information: http://vger.kernel.org/majordomo-info.html * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [PATCH] http: add custom hostname to IP address resolves' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Code repositories for project(s) associated with this inbox: https://80x24.org/mirrors/git.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).