gvfs-helper: override cache server for post

This extension of the gvfs.cache-server config now allows a new key,
gvfs.post.cache-server, to overrid the cache-server URL for only the
batched objects endpoint.

The purpose of this config is to allow for incremental testing and
deployment of new cache-server infrastructure.

Signed-off-by: Derrick Stolee <stolee@gmail.com>

Author: derrickstolee

Documentation/config/gvfs.adoc

@@ -13,6 +13,9 @@ gvfs.<verb>.cache-server::
 	get::
 		Use this cache server when downloading objects immediately via the
 		GET endpoint.
+	post::
+		Use this cache server when downloading objects in batches using the
+		POST endpoint.
 --
 
 gvfs.sharedcache::

gvfs-helper.c

@@ -396,6 +396,7 @@ enum gh__server_type {
 enum gh__verb {
 	PREFETCH,
 	GET,
+	POST,
 };
 
 static void update_cache_server_for_verb(enum gh__verb verb)
@@ -413,6 +414,10 @@ static void update_cache_server_for_verb(enum gh__verb verb)
 			verbstr = "get";
 			break;
 
+		case POST:
+			verbstr = "post";
+			break;
+
 		default:
 			gh__global.cache_server_url_backup = NULL;
 			return;
@@ -3439,7 +3444,9 @@ static void do__http_post__gvfs_objects(struct gh__response_status *status,
 
 	setup_gvfs_objects_progress(&params, j_pack_num, j_pack_den);
 
+	update_cache_server_for_verb(POST);
 	do_req__with_fallback("gvfs/objects", &params, status);
+	reset_cache_server();
 
 	gh__request_params__release(&params);
 	jw_release(&jw_req);

t/t5799-gvfs-helper.sh

@@ -1773,4 +1773,53 @@ test_expect_success 'verb-specific cache-server: prefetch does NOT use gvfs.get.
 	verify_server_was_not_contacted 1
 '
 
+test_expect_success 'verb-specific cache-server: post uses gvfs.post.cache-server' '
+	test_when_finished "per_test_cleanup" &&
+	test_when_finished "git -C \"$REPO_T1\" config --unset gvfs.post.cache-server" &&
+	start_gvfs_protocol_server 0 &&
+	start_gvfs_protocol_server 1 &&
+
+	# Configure server 0 as default cache-server and server 1 for post.
+	git -C "$REPO_T1" config gvfs.cache-server "$(cache_server_url 0)" &&
+	git -C "$REPO_T1" config gvfs.post.cache-server "$(cache_server_url 1)" &&
+
+	# Run post - should go to server 1.
+	git -C "$REPO_T1" gvfs-helper \
+		--cache-server=trust \
+		--remote=origin \
+		--no-progress \
+		post \
+		<"$OIDS_BLOBS_FILE" >OUT.output 2>OUT.stderr &&
+
+	# Verify server 1 was contacted (post-specific).
+	verify_server_was_contacted 1 &&
+
+	# Verify server 0 was NOT contacted.
+	verify_server_was_not_contacted 0
+'
+
+test_expect_success 'verb-specific cache-server: get does NOT use gvfs.post.cache-server' '
+	test_when_finished "per_test_cleanup" &&
+	test_when_finished "git -C \"$REPO_T1\" config --unset gvfs.post.cache-server" &&
+	start_gvfs_protocol_server 0 &&
+	start_gvfs_protocol_server 1 &&
+
+	# Configure server 0 as default cache-server and server 1 for post.
+	git -C "$REPO_T1" config gvfs.cache-server "$(cache_server_url 0)" &&
+	git -C "$REPO_T1" config gvfs.post.cache-server "$(cache_server_url 1)" &&
+
+	# Run get - should go to server 0 (default), not server 1 (post).
+	git -C "$REPO_T1" gvfs-helper \
+		--cache-server=trust \
+		--remote=origin \
+		get \
+		<"$OID_ONE_BLOB_FILE" >OUT.output 2>OUT.stderr &&
+
+	# Verify server 0 was contacted (default cache-server).
+	verify_server_was_contacted 0 &&
+
+	# Verify server 1 was NOT contacted (post-specific).
+	verify_server_was_not_contacted 1
+'
+
 test_done