bytecodealliance / bytecodealliance/wasm-tools

wave: Require escaping some "problematic" unicode characters

Open
#1,629 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
1.8k
Forks
351
Avg merge
16h 57m
Merged PRs (30d)
38

Description

Currently char and string escapes are only required for: `\n`, `\\` and either `\'` (chars) or `\"` (strings).

In order to avoid human parsing problems, consider a few other mandatory escapes:
* bidi chars (source of CVEs)
* unicode deprecations

See: https://github.com/bytecodealliance/wac/blob/b294ae04cd85f12a619db08165b3117b8f977b0d/crates/wac-parser/src/lexer.rs#L46-L82

(migrated from https://github.com/lann/wasm-wave/issues/31)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.