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

Ignore more proxy headers #4

Merged
merged 1 commit into from
Jan 8, 2016
Merged

Conversation

dochang
Copy link

@dochang dochang commented Jan 8, 2016

57b9c7a just removes the header
returned by Squid.  But some proxies -- such as [polipo][] -- return a
different header.  Remove those headers as well.

[polipo]: https://github.com/jech/polipo/blob/master/tunnel.c#L302
@abingham
Copy link
Owner

abingham commented Jan 8, 2016

Thanks!

abingham pushed a commit that referenced this pull request Jan 8, 2016
@abingham abingham merged commit 48a3596 into abingham:master Jan 8, 2016
@dochang dochang deleted the skip-proxy-headers branch January 8, 2016 08:55
@dochang
Copy link
Author

dochang commented Jan 8, 2016

@abingham Thank you for adopting request. I saw you have migrated the recipe in melpa. But request-deferred recipe also needs to be migrated. Also, I suggest you migrate the recipe in el-get too.

@abingham
Copy link
Owner

abingham commented Jan 8, 2016

Honestly I'm trying to make as few changes as possible. I eventually want @tkf to take it back over. Is there a strong technical reason to migrate request-deferred right now?

And good point about el-get. I've suggested that they update their recipe.

@abingham
Copy link
Owner

abingham commented Jan 8, 2016

Ah, wait, I see what you're saying about request-deferred...I had forgotten that it's part of the same repo! I'll update the melpa recipe. Thanks.

@abingham
Copy link
Owner

abingham commented Jan 8, 2016

I've submitted a PR to melpa redirecting request-deferred to this repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants