Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix some of the proxy code paths #141

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix some of the proxy code paths #141

wants to merge 3 commits into from

Conversation

daurnimator
Copy link
Owner

@daurnimator daurnimator commented Mar 6, 2019

Blocked on figuring out what to send to a HTTP 2 proxy in :path (absolute-form? or not?)
See httpwg/http2-spec#768

Related:

@daurnimator daurnimator added the bug label Mar 6, 2019
@daurnimator daurnimator force-pushed the proxy-fixes branch 2 times, most recently from 2356f9c to 86e6fcd Compare March 6, 2019 23:03
@coveralls
Copy link

coveralls commented Mar 7, 2019

Coverage Status

Coverage decreased (-0.005%) to 86.832% when pulling 53d0596 on proxy-fixes into bc49a4b on master.

  - absolute-form host overrides host header
  - CONNECT requests should have a host header
  - OPTIONS requests in absolute-form without a path should have path '*'
@daurnimator daurnimator force-pushed the proxy-fixes branch 2 times, most recently from c09e94f to 1f15d3f Compare March 13, 2019 06:38
@daurnimator daurnimator added this to the 0.4 milestone Apr 28, 2019
@daurnimator daurnimator removed this from the 0.4 milestone Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants