1. Added escape sequence support: Modified the header parsing logic… #11
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
… to handle escaped commas (,) in header values
- Simple headers without commas
- Headers with escaped commas (like Keep-Alive)
- Multiple escaped commas in a single value
- Backslashes that aren't escape sequences
- Edge cases with whitespace and trailing commas
The solution allows users to include commas in header values by escaping them with a backslash. For example:
-e CUSTOM_HEADERS="Connection:keep-alive,Keep-Alive:timeout=5\,max=200"
This will correctly parse as two headers: