3 ms·
They're not doing arbitrary parsing, they just need the ability to un-escape XML characters for a particular case. I'd be shocked if (after all the abstraction)
by ninkendo 5y ago
They're not doing arbitrary parsing, they just need the ability to un-escape XML characters for a particular case. I'd be shocked if (after all the abstraction) the `he` library didn't just have a single function which does that. I'd prefer just seeing how `he` does it and implementing it myself.
Moreover, they're already using a `fast-xml-parser` for doing XML parsing. Presumably it doesn't have an unescape function, so they're taking on a dependency on another XML parser (and keeping the old one!) just to get the one function.
- paulddraper 5y agohe is for HTML(/XML) encoding. It's kinda dumb that fast-xml-parser doesn't full parse the XML and leave content+attribute values in raw forms. The docs for fast-xml-parser show how to combine it and he. Had they done that, they wouldn't have this bug that completely breaks the SQS client for me.