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

Allow passing custom configuration to parse_headers #126

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gootorov
Copy link

There are several protocols that look almost identical to HTTP (in my case, it is SIP). Often the only difference is the request-line and/or the response-line. In SIP in particular, multiline headers are the norm (allow_obsolete_multiline_headers_in_responses).

Being able to parse the request/response-line by myself and then using parse_headers with a custom configuration would be very convenient.

@gootorov
Copy link
Author

Friendly ping! :)
If you're unable to review for the time being, that's totally fine of course. Only making sure this isn't forgotten.

@gootorov
Copy link
Author

gootorov commented Dec 28, 2022

@seanmonstar Apologies for pinging again. Do you mind having a look at this PR? This is a small, non-breaking change. Having this merged would be really great, if possible.

If there's something that needs to be changed, such as documentation, tests, etc -- please let me know.

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.

1 participant