delta/curl.git/lib/http2.c, branch bagder/test493-https github.com: bagder/curl.git urldata: merge "struct DynamicStatic" into "struct UrlState" 2021-03-26T22:19:20+00:00 Daniel Stenberg daniel@haxx.se 2021-03-26T13:25:45+00:00 95cbcec8f986492766c4be3922af1e7644e1e7c5 Both were used for the same purposes and there was no logical separation between them. Combined, this also saves 16 bytes in less holes in my test build. Closes #6798 
Both were used for the same purposes and there was no logical separation
between them. Combined, this also saves 16 bytes in less holes in my
test build.

Closes #6798
http2: don't set KEEP_SEND when there's no more data to be sent 2021-03-15T22:29:35+00:00 Robert Ronto rronto@gmail.com 2021-03-15T14:35:10+00:00 068661bb028d7c9f08e5d35cf877c9584f9a46e5 this should fix an issue where curl sometimes doesn't send out a request with authorization info after a 401 is received over http2 Closes #6747 
this should fix an issue where curl sometimes doesn't send out a request
with authorization info after a 401 is received over http2

Closes #6747
http2: fail if connection terminated without END_STREAM 2021-03-12T16:54:39+00:00 oxalica oxalicc@pm.me 2021-03-12T13:11:08+00:00 d1f40078c13e85c56332dcb7f908fe2a7b65eb22 Closes #6736 
Closes #6736
http2: remove conn->data use 2021-02-15T15:33:53+00:00 Daniel Stenberg daniel@haxx.se 2021-02-15T07:35:32+00:00 723c0e75fa3e74cdad5ad2b3c130f9b974e89ef1 ... but instead use a private alternative that points to the "driving transfer" from the connection. We set the "user data" associated with the connection to be the connectdata struct, but when we drive transfers the code still needs to know the pointer to the transfer. We can change the user data to become the Curl_easy handle, but with older nghttp2 version we cannot dynamically update that pointer properly when different transfers are used over the same connection. Closes #6520 
... but instead use a private alternative that points to the "driving
transfer" from the connection. We set the "user data" associated with
the connection to be the connectdata struct, but when we drive transfers
the code still needs to know the pointer to the transfer. We can change
the user data to become the Curl_easy handle, but with older nghttp2
version we cannot dynamically update that pointer properly when
different transfers are used over the same connection.

Closes #6520
urldata: remove duplicate 'upkeep_interval_ms' from connectdata 2021-01-27T08:19:08+00:00 Daniel Stenberg daniel@haxx.se 2021-01-26T13:23:36+00:00 1c3def53c71f85bb3ae16cd4c39dbdcc2a6c460f ... and rely only on the value already set in Curl_easy. Closes #6534 
... and rely only on the value already set in Curl_easy.

Closes #6534
lib: pass in 'struct Curl_easy *' to most functions 2021-01-17T22:56:09+00:00 Daniel Stenberg daniel@haxx.se 2021-01-08T16:58:15+00:00 215db086e09665ee7af9b646ad6c4d6e281001ac ... in most cases instead of 'struct connectdata *' but in some cases in addition to. - We mostly operate on transfers and not connections. - We need the transfer handle to log, store data and more. Everything in libcurl is driven by a transfer (the CURL * in the public API). - This work clarifies and separates the transfers from the connections better. - We should avoid "conn->data". Since individual connections can be used by many transfers when multiplexing, making sure that conn->data points to the current and correct transfer at all times is difficult and has been notoriously error-prone over the years. The goal is to ultimately remove the conn->data pointer for this reason. Closes #6425 
... in most cases instead of 'struct connectdata *' but in some cases in
addition to.

- We mostly operate on transfers and not connections.

- We need the transfer handle to log, store data and more. Everything in
  libcurl is driven by a transfer (the CURL * in the public API).

- This work clarifies and separates the transfers from the connections
  better.

- We should avoid "conn->data". Since individual connections can be used
  by many transfers when multiplexing, making sure that conn->data
  points to the current and correct transfer at all times is difficult
  and has been notoriously error-prone over the years. The goal is to
  ultimately remove the conn->data pointer for this reason.

Closes #6425
failf: remove newline from formatting strings 2020-12-25T22:42:16+00:00 Daniel Stenberg daniel@haxx.se 2020-12-23T22:41:13+00:00 aba01da6390894c9e7ebb6691da544bc01b6324d ... as failf adds one itself. Also: add an assert() to failf() that triggers on a newline in the format string! Closes #6365 
... as failf adds one itself.

Also: add an assert() to failf() that triggers on a newline in the
format string!

Closes #6365
h2: do not wait for RECV on paused transfers 2020-12-22T11:44:18+00:00 Daniel Stenberg daniel@haxx.se 2020-12-22T08:09:46+00:00 c7f95fa0cc252167cb210e1c0ab4f8bbabc1c371 ... as the socket might be readable all the time when paused and thus causing a busy-loop. Reported-by: Harry Sintonen Reviewed-by: Jay Satiro Fixes #6356 Closes #6357 
... as the socket might be readable all the time when paused and thus
causing a busy-loop.

Reported-by: Harry Sintonen
Reviewed-by: Jay Satiro
Fixes #6356
Closes #6357
infof/failf calls: fix format specifiers 2020-11-24T12:18:41+00:00 Rikard Falkeborn rikard.falkeborn@gmail.com 2020-11-23T21:42:07+00:00 920f49a20be51170ec1804f2736ea3e2593cdbe5 Update a few format specifiers to match what is being printed. Closes #6241 
Update a few format specifiers to match what is being printed.

Closes #6241
urldata: remove 'void *protop' and create the union 'p' 2020-11-23T15:16:16+00:00 Daniel Stenberg daniel@haxx.se 2020-11-23T07:32:41+00:00 a95a6ce6b809693a1195e3b4347a6cfa0fbc2ee7 ... to avoid the use of 'void *' for the protocol specific structs done per transfer. Closes #6238 
... to avoid the use of 'void *' for the protocol specific structs done
per transfer.

Closes #6238