Skip to content

Conversation

@cookie-s
Copy link
Contributor

@cookie-s cookie-s commented Nov 24, 2024

This PR fixes CRLF replacement.

References:

https://datatracker.ietf.org/doc/html/rfc7230#section-1.2

The following core rules are included by reference, as defined in
[RFC5234], Appendix B.1: ALPHA (letters), CR (carriage return), CRLF
(CR LF), CTL (controls), DIGIT (decimal 0-9), DQUOTE (double quote),
HEXDIG (hexadecimal 0-9/A-F/a-f), HTAB (horizontal tab), LF (line
feed), OCTET (any 8-bit sequence of data), SP (space), and VCHAR (any
visible [USASCII] character).

https://datatracker.ietf.org/doc/html/rfc5234#appendix-B.1

CR             =  %x0D
                                ; carriage return
CRLF           =  CR LF
                                ; Internet standard newline
:
:
LF             =  %x0A
                                ; linefeed

https://doc.rust-lang.org/reference/tokens.html#ascii-escapes

\n | Newline
\r | Carriage return

@d0iasm
Copy link
Owner

d0iasm commented Dec 2, 2024

Thank you for finding and fixing this issue!

@d0iasm d0iasm merged commit 8059f0e into d0iasm:main Dec 2, 2024
2 checks passed
@cookie-s cookie-s deleted the fix-crlf branch December 4, 2024 02:11
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.

2 participants