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

Add parameters to suppress sending X-Powered-By header #54

Closed
wants to merge 3 commits into from

Conversation

vfsoraki
Copy link

It is related to issue #36

@clue
Copy link
Member

clue commented Sep 13, 2016

Thanks for your PR, but as much as I link this feature, I'm currently having to vote 👎 on this purely due to how much new code this introduces and how many tests this would require.

As an alternative, I would suggest we tackle this as part of #28, where we can likely introduce some kind of default response which would default to having this header applied.

@clue clue changed the title Thelastblack/master Add parameters to suppress sending X-Powered-By header Feb 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants