Skip to content

Conversation

486
Copy link
Contributor

@486 486 commented Sep 29, 2014

"The sequence
CRLF (0x0d0a) is used to end a record, although parsers SHOULD be
tolerant and also accept records terminated with a single newline
character."

I have a commercial transcoding gateway here that spits out SDP with \n termination. I think it would be good to handle that directly.

"The sequence
CRLF (0x0d0a) is used to end a record, although parsers SHOULD be
tolerant and also accept records terminated with a single newline
character."
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 103eb27 on 486:newline into 595a525 on clux:master.

@clux
Copy link
Owner

clux commented Sep 29, 2014

Yeah, thank you. I agree, easy to handle this here.
Nice way you've handled this with the regexp split :]

Will merge and publish a major (mostly because there's no reason to stay in 0.x land anymore) later today.

clux added a commit that referenced this pull request Sep 29, 2014
accept all newline variations
@clux clux merged commit d1e538d into clux:master Sep 29, 2014
clux added a commit that referenced this pull request Sep 30, 2014
@clux
Copy link
Owner

clux commented Sep 30, 2014

sdp-transform@1.0.0 published.

@486
Copy link
Contributor Author

486 commented Oct 1, 2014

Very nice, thanks for all your effort!

@486 486 deleted the newline branch October 1, 2014 08:56
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.

3 participants