search

braintree / sanitize-url

MIT License
307 stars 35 forks source link

Should sanitize-url replace zero width characters? #35

Closed jackw closed 3 years ago

jackw commented 3 years ago

Having updated to v5.0.1 I've noticed that sanitize-url is no longer replacing zero width characters.

Is this by design or a regression caused by changes made in v5.0.1?

Please see https://github.com/grafana/grafana/issues/34528 for an example.

crookedneighbor commented 3 years ago

Yeah, I believe that's a regression. We'll get it fixed.

jackw commented 3 years ago

Amazing! Thanks so much @crookedneighbor

crookedneighbor commented 3 years ago

This is out in v5.0.2