[curl/curl] 252790: http2: call the handle-closed function correctly o...

  Branch: refs/heads/master
  Home:   https://github.com/curl/curl
  Commit: 252790c5335a221179dfbacb40e065750bbe7544
      https://github.com/curl/curl/commit/252790c5335a221179dfbacb40e065750bbe7544
  Author: Daniel Stenberg <[email protected]>
  Date:   2021-04-21 (Wed, 21 Apr 2021)

  Changed paths:
    M lib/http2.c

  Log Message:
  -----------
  http2: call the handle-closed function correctly on closed stream

This was this one condition where the stream could be closed due to an
error and the function would still wrongly just return 0 for it.

Reported-by: Gergely Nagy
Fixes #6862
Closes #6910


  Commit: 605e8423554e6c53c7755805f18354973b563b16
      https://github.com/curl/curl/commit/605e8423554e6c53c7755805f18354973b563b16
  Author: Daniel Stenberg <[email protected]>
  Date:   2021-04-21 (Wed, 21 Apr 2021)

  Changed paths:
    M lib/http.h
    M lib/http2.c
    M lib/http2.h
    M lib/multi.c

  Log Message:
  -----------
  http2: move the stream error field to the per-transfer storage

Storing a stream error in the per-connection struct was an error that lead to
race conditions as subsequent stream handling could overwrite the error code
before it was used for the stream with the actual problem.

Closes #6910


  Commit: 9c18c0b4dd8f00ed60f0c1994138afe556428114
      https://github.com/curl/curl/commit/9c18c0b4dd8f00ed60f0c1994138afe556428114
  Author: Daniel Stenberg <[email protected]>
  Date:   2021-04-21 (Wed, 21 Apr 2021)

  Changed paths:
    M lib/multi.c

  Log Message:
  -----------
  multi: don't close connection HTTP_1_1_REQUIRED

The ConnectionExists() function will note that the new transfer wants
less then h2 and that it can't multiplex it and therefor opt to open a
new connection instead.


  Commit: 9cb48457c6a9db533d6759a31a49ba13f1bc591b
      https://github.com/curl/curl/commit/9cb48457c6a9db533d6759a31a49ba13f1bc591b
  Author: Daniel Stenberg <[email protected]>
  Date:   2021-04-21 (Wed, 21 Apr 2021)

  Changed paths:
    M lib/url.c

  Log Message:
  -----------
  ConnectionExists: respect requests for h1 connections better

... for situations when multiplexing isn't enabled on the h2 connection
and h1 is explicitly requested for the transfer.

Assisted-by: Gergely Nagy


Compare: https://github.com/curl/curl/compare/6b97f1311a14...9cb48457c6a9
_______________________________________________
curl-commits mailing list
[email protected]
https://cool.haxx.se/cgi-bin/mailman/listinfo/curl-commits