squash! gvfs-helper: expose gvfs/objects GET and POST semantics

For the t5799.13(no server) test case, we specifically work around a
change of behavior in cURL v8.16.0 where the generous default connection
timeout of 5 minutes started to be applied several times in instances
where a connection is refused. The work-around is to add the
`--connect-timeout-ms` option and set it really low in that test case.
Unfortunately, with that work-around that cURL version will report a
timeout instead of a failure to connect. See
curl/curl#18767 for details.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>

Author: dscho

gvfs-helper.c

@@ -381,6 +381,7 @@ static struct gh__global {
 	int cache_server_is_initialized; /* did sub-command look for one */
 	int main_creds_need_approval; /* try to only approve them once */
 
+	unsigned long connect_timeout_ms;
 } gh__global;
 
 enum gh__server_type {
@@ -2970,6 +2971,10 @@ static void do_req(const char *url_base,
 		curl_easy_setopt(slot->curl, CURLOPT_NOPROGRESS, 1L);
 	}
 
+	if (gh__global.connect_timeout_ms)
+		curl_easy_setopt(slot->curl, CURLOPT_CONNECTTIMEOUT_MS,
+				gh__global.connect_timeout_ms);
+
 	gh__run_one_slot(slot, params, status);
 	strbuf_release(&rest_url);
 }
@@ -3678,6 +3683,9 @@ static enum gh__error_code do_sub_cmd__get(int argc, const char **argv)
 	static struct option get_options[] = {
 		OPT_INTEGER('r', "max-retries", &gh__cmd_opts.max_retries,
 			    N_("retries for transient network errors")),
+		OPT_UNSIGNED(0, "connect-timeout-ms",
+			     &gh__global.connect_timeout_ms,
+			     N_("try to connect only for this many milliseconds")),
 		OPT_END(),
 	};
 

t/t5799-gvfs-helper.sh

@@ -850,9 +850,25 @@ mayhem_observed__close () {
 	fi
 }
 
+test_lazy_prereq CURL_8_16_0 '
+	git gvfs-helper curl-version = 8.16.0 ||
+	test 8.15.0-DEV = "$(git gvfs-helper curl-version)"
+'
+
 test_expect_success 'curl-error: no server' '
 	test_when_finished "per_test_cleanup" &&
 
+	connect_timeout_ms= &&
+	# CURLE_COULDNT_CONNECT 7
+	regex="error: get: (curl:7)" &&
+	if test_have_prereq CURL_8_16_0
+	then
+		connect_timeout_ms=--connect-timeout-ms=200 &&
+		# CURLE_COULDNT_CONNECT 7
+		# CURLE_OPERATION_TIMEDOUT 28
+		regex="error: get: (curl:\(7\|28\))"
+	fi &&
+
 	# Try to do a multi-get without a server.
 	#
 	# Use small max-retry value because of exponential backoff,
@@ -864,10 +880,9 @@ test_expect_success 'curl-error: no server' '
 		--remote=origin \
 		get \
 		--max-retries=2 \
+		$connect_timeout_ms \
 		<"$OIDS_FILE" >OUT.output 2>OUT.stderr &&
-
-	# CURLE_COULDNT_CONNECT 7
-	test_grep "error: get: (curl:7)" OUT.stderr
+	test_grep "$regex" OUT.stderr
 '
 
 test_expect_success 'curl-error: close socket while reading request' '