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 Message::getRawHeaders() to allow getting the original case of headers #12

Closed
wants to merge 1 commit into from

Conversation

@nicolas-grekas
Copy link
Contributor

nicolas-grekas commented Jan 7, 2020

A proposal to fix amphp/http-client#242
We use a similar strategy in Symfony for the same need.

@nicolas-grekas

This comment has been minimized.

Copy link
Contributor Author

nicolas-grekas commented Jan 7, 2020

I missed #11 before submitting...
Since the implementation is different, this is some food for thoughts :)

@nicolas-grekas nicolas-grekas force-pushed the nicolas-grekas:raw-header branch from d5937af to b8f2e05 Jan 7, 2020
@kelunik

This comment has been minimized.

Copy link
Member

kelunik commented Jan 12, 2020

Closing in favor of #11. Thanks!

@kelunik kelunik closed this Jan 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.