Re: [PATCH v5 10/10] credential: add WWW-Authenticate header to cred requests

From: Derrick Stolee <derrickstolee@github.com>
Date: Jan 12, 2023, 20:41 UTC
Message-ID: <98940e93-c4c5-01ec-54b2-b6015f488ad0@github.com>
In-Reply-To: <af66d2d2ede2a502f32d74c86f302598c68d1476.1673475190.git.gitgitgadget@gmail.com>

On 1/11/2023 5:13 PM, Matthew John Cheetham via GitGitGadget wrote:

Show 6 quoted lines

> +static void credential_write_strvec(FILE *fp, const char *key,
> +				    const struct strvec *vec)
> +{
> +	int i = 0;
> +	const char *full_key = xstrfmt("%s[]", key);
> +	for (; i < vec->nr; i++) {

style nit: use "int i;" and "for (i = 0; ..."

Show 10 quoted lines

>  test_expect_success 'http auth anonymous no challenge' '
>  	test_when_finished "per_test_cleanup" &&
> -	start_http_server &&
> +
> +	cat >auth.config <<-EOF &&
> +	[auth]
> +	    allowAnonymous = true
> +	EOF
> +
> +	start_http_server --auth-config="$TRASH_DIRECTORY/auth.config" &&

I see that you added auth.allowAnonymous and --auth-config options in Patch 6, so perhaps this test change could move to that patch.

Thanks, -Stolee