rack-utf8_sanitizer
rack-utf8_sanitizer copied to clipboard
Remove '\u0000' from input when sanitizing null input
There's a lot of unrelated formatting chances so I won't reviewing the PR.
@whitequark Aside from the formatting changes, do you have an issue with this change? Would like to use this gem for sanitizing but we also need to sanitize \u0000, and happy to put up a PR without needless style changes
I haven't reviewed it due to the formatting changes. I haven't even used Ruby in almost a decade and I don't want to waste my time looking at a PR like this one.
I haven't even used Ruby in almost a decade and I don't want to waste my time looking at a PR like this one.
It's OK, but maybe you should assign a new maintainer for this repo. If you're not sure who — open an issue for this.
I'm aware of my options.
@whitequark / @AlexWayfer - the PR has been updated to remove unnecesarry formatting changes. Can we get a review please?
I understand the stated purpose, but I don't think this should be the responsibility of this library, and I don't think removing
\\u0000is even correct in the general case. (It is perfectly valid UTF-8, after all.)
even if it was optional as well?
This has nothing to do with invalid UTF-8, so yes.