braintree / sanitize-url

MIT License
312 stars 35 forks source link

feat: Add explicit check for null/undefined and throw Error #11

Closed jLouzado closed 5 years ago

jLouzado commented 5 years ago

Related to discussion here: https://github.com/braintree/sanitize-url/pull/10#issuecomment-450802856

crookedneighbor commented 5 years ago

I think it's fine as is. The main point of this lib is to strip out characters that would inject javascript. I'm fine with just converting it to about:blank when no url is provided.