Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Proxy connection headers not checked for as standard #914

Closed
pettzilla1 opened this issue May 22, 2024 · 1 comment
Closed

Proxy connection headers not checked for as standard #914

pettzilla1 opened this issue May 22, 2024 · 1 comment

Comments

@pettzilla1
Copy link

connect_headers = merge_headers(

Hi I've noticed that headers which can change during proxy auth aren't checked for to see if they are in the request, I think it may be worth adding a check to see if ('proxy-authorization','proxy-connection','connection') are in the request and need to be added to the proxy headers,
If this can already be done happy to be pointed in the right direction

@tomchristie
Copy link
Member

I'll bump this into "discussion" as a starting point, as I don't think it's yet clearly framed enough to warrant an issue.

@encode encode locked and limited conversation to collaborators Jun 11, 2024
@tomchristie tomchristie converted this issue into discussion #925 Jun 11, 2024

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants