YAML with emoji fails about yaml-js

[connec]

Interesting! If you post the YAML document that's failing I'll look into it when I can 😄

[syropian]

Sure! You can grab it here http://d.pr/f/1kVn3

[klokane]

Hi, we have same problem with unicode char U+1F60E (Smiling face with sunglases). It comes from external source, but it cripple our processing tool.

I investigate for reason.

It is because https://github.com/connec/yaml-js/blob/master/src/lib/reader.coffee#L19

There is check for NON_PRINTABLE character set. But there is explicit range for allowed chars.
If I comment calling @check_printable() it is parsed correctly.

So I propose instead of checking for "allowed set" of unicode chars, check just for a few dangerous char, like \u2028 or \u2029

Is it acceptable solution for you @connec? I'm able to delivery PR in this case

Cheers

[pksunkara]

@connec Can we get a comment on this?

[connec]

Sorry for the delay investigating this. I tweaked the NON_PRINTABLE regular expression to allow code points above 0xFFFF. For reference I tweaked a regex created by regenerate.

[pksunkara]

@connec Can we get a npm release please?

[connec]

@pksunkara published 0.1.5!