On Mon, Sep 25, 2023 at 7:58 AM <rsbecker@xxxxxxxxxxxxx> wrote: > > On Sunday, September 24, 2023 7:40 PM, Jiang Xin wrote: > >On Sun, Sep 24, 2023 at 2:52 PM Eric Sunshine <sunshine@xxxxxxxxxxxxxx> > >wrote: > >> > >> On Sat, Sep 23, 2023 at 11:22 AM Jiang Xin <worldhello.net@xxxxxxxxx> wrote: > >> > Even though we can establish a stateless connection, we still cannot > >> > archive the remote repository using a stateless HTTP protocol. Try > >> > the following steps to make it work. > >> > [...] > >> > Signed-off-by: Jiang Xin <zhiyou.jx@xxxxxxxxxxxxxxx> > >> > --- > >> > diff --git a/http-backend.c b/http-backend.c @@ -639,10 +640,19 @@ > >> > static void check_content_type(struct strbuf *hdr, const char *accepted_type) > >> > - const char *argv[] = {NULL, "--stateless-rpc", ".", NULL}; > >> > + const char *argv[4]; > >> > > >> > + if (!strcmp(service_name, "git-upload-archive")) { > >> > + argv[1] = "."; > >> > + argv[2] = NULL; > >> > + } else { > >> > + argv[1] = "--stateless-rpc"; > >> > + argv[2] = "."; > >> > + argv[3] = NULL; > >> > + } > >> > >> It may not be worth a reroll, but since you're touching this code > >> anyhow, these days we'd use `strvec` for this: > >> > >> struct strvec argv = STRVEC_INIT; > >> if (strcmp(service_name, "git-upload-archive")) > >> strvec_push(&argv, "--stateless-rpc"); > >> strvec_push(&argv, "."); > > > >Good suggestion, I'll queue this up as part of next reroll. > > Which test covers this change? See: https://lore.kernel.org/git/20230923152201.14741-4-worldhello.net@xxxxxxxxx/#Z31t:t5003-archive-zip.sh > Thanks, > Randall >