On Fri, Jul 23, 2021 at 05:28:33PM +0200, Sebastian Benoit wrote: > Job Snijders(j...@openbsd.org) on 2021.07.23 15:23:49 +0000: > > Hi all, > > > > Based on suggestions from Julian Reschke. > > > > * "Connection: keep-alive" isn't needed, as the HTTP 1.1 default is to > > use persistent connections (RFC 7230, section 6.3). > > > > * "Host" is recommended to be in the front. > > > > * "Accept-Encoding: identity" makes it clear to the server compression > > encodings are not supported. > > https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Accept-Encoding > > > > OK? > > reasonable. > ok benno@
Agreed, I think the time of possible 'Connection: close' is over. The spec for Accept-Encoding is whako, the default is no-preference and therefor any encoding is OK. Who came up with such a bad default? Because of this OK claudio@ > > > > Kind regards, > > > > Job > > > > Index: http.c > > =================================================================== > > RCS file: /cvs/src/usr.sbin/rpki-client/http.c,v > > retrieving revision 1.33 > > diff -u -p -r1.33 http.c > > --- http.c 10 May 2021 15:12:33 -0000 1.33 > > +++ http.c 23 Jul 2021 15:13:35 -0000 > > @@ -847,9 +847,10 @@ http_request(struct http_connection *con > > conn->bufpos = 0; > > if ((r = asprintf(&conn->buf, > > "GET /%s HTTP/1.1\r\n" > > - "Connection: keep-alive\r\n" > > + "Host: %s\r\n" > > + "Accept-Encoding: identity\r\n" > > "User-Agent: " HTTP_USER_AGENT "\r\n" > > - "Host: %s\r\n%s\r\n", > > + "%s\r\n", > > epath, host, > > modified_since ? modified_since : "")) == -1) > > err(1, NULL); > > > -- :wq Claudio
