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 #30

Closed
wants to merge 11 commits into from
Closed

Conversation

dochang
Copy link

@dochang dochang commented Nov 26, 2015

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

@abingham
Copy link
Contributor

abingham commented Jan 3, 2016

@dochang I'm in the process of updating MELPA to use a new repository since @tkf is no maintaining this one. Please submit this pull-request to https://github.com/abingham/emacs-request/.

abingham and others added 6 commits January 3, 2016 17:22
Use cl-lib instead of the old prefix-less CL package
There is no '&allow-other-keys&rest' keyword and it is treated as
normal argument. And '&rest' keyword should be declared before '&key'
keyword.
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
@dochang
Copy link
Author

dochang commented Jan 8, 2016

Closed. See abingham#4

@dochang dochang closed this Jan 8, 2016
@dochang dochang deleted the skip-proxy-headers branch January 8, 2016 08:55
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.

5 participants