commit 767da05cecb82ef3e8fe53a8bb977e4fd054a43b
from: Omar Polo <op@omarpolo.com>
date: Fri Aug 13 07:48:19 2021 UTC

fix handling of query part in gopher URLs

The computed request was discarded because we passed `path' to
make_request.  Passing NULL here makes the net proc use our computed
request string.  There was an error in the computed request btw, where
\r\n were strcpy'd instead of cat'd.

commit - 6c739557ef552609512f2d7d7cf8ce42600b76d7
commit + 767da05cecb82ef3e8fe53a8bb977e4fd054a43b
blob - d2223ed921f396012bb4c6436115abc2c3920096
blob + 322247c14091b63b4dd347083ffe695d0510cc9b
--- telescope.c
+++ telescope.c
@@ -669,9 +669,9 @@ load_gopher_url(struct tab *tab, const char *url)
 		strlcat(req.req, "?", sizeof(req.req));
 		strlcat(req.req, tab->uri.query, sizeof(req.req));
 	}
-	strlcpy(req.req, "\r\n", sizeof(req.req));
+	strlcat(req.req, "\r\n", sizeof(req.req));
 
-	return make_request(tab, &req, PROTO_GOPHER, path);
+	return make_request(tab, &req, PROTO_GOPHER, NULL);
 }
 
 static int