Skip to content

Fix https://github.com/apihlaja/node-irsdk/issues/80 - #84

Open
kenklin wants to merge 1 commit into
apihlaja:masterfrom
kenklin:master
Open

kenklin wants to merge 1 commit into
apihlaja:masterfrom
kenklin:master

Conversation

@kenklin

@kenklin kenklin commented Apr 15, 2020

Copy link
Copy Markdown

Fixes #80 where YAML correctly rejects control characters in the SessionInfo. I've encountered UTF-8 C2 9F (Unicode 00 9F) several times.

@sabifa

sabifa commented Apr 23, 2020

Copy link
Copy Markdown

Any chance you could review the changes? @apihlaja 🙂

@sabifa

sabifa commented Sep 28, 2020

Copy link
Copy Markdown

I've just experienced this parsing error and I can confirm that this change fixes the problem!

This branch has not been deployed

No deployments
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.

there is a character encoding issue. Could you please fix it?

2 participants