Skip to content

HTTPS-Proxy: don't offer h2 for https proxy conn. draft1#1546

Closed
lon990109tim wants to merge 1 commit into
curl:masterfrom
jay:disable_h2_for_https_proxy
Closed

HTTPS-Proxy: don't offer h2 for https proxy conn. draft1#1546
lon990109tim wants to merge 1 commit into
curl:masterfrom
jay:disable_h2_for_https_proxy

Conversation

@lon990109tim

Copy link
Copy Markdown

Bug: #1254

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.03%) to 73.186% when pulling 76e3034 on jay:disable_h2_for_https_proxy into de0adda on curl:master.

@bagder bagder left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fine with this, are you @jay ? I realize there's room for improvement to do this at the generic vtls level, but this is still better than what we have right now IMHO.

@bagder bagder closed this in 17c5d05 Jun 16, 2017
@lock lock Bot locked as resolved and limited conversation to collaborators Jan 18, 2019
@jay
jay deleted the disable_h2_for_https_proxy branch November 27, 2022 21:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Development

Successfully merging this pull request may close these issues.

4 participants