deprecate / metal-uri

Class for parsing and formatting URIs.
Other
3 stars 7 forks source link

Fallback doesn't not throw errors when expected #11

Closed fernandosouza closed 7 years ago

fernandosouza commented 7 years ago

Hey, guys. I would like to bring here a discussion I had with @brunobasto.

The fallback used in Metal URI doesn't behave as expected. Differently of the main implementation, the fallback doesn't throw errors.

One example of this is when the url port exceeds the maximum allowed 65535:

For the given url http://localhost:99999 the following happens:

There is a pull request for SennaJS to prevent crashes because of errors from Metal URI, but the test fails in browsers that use the fallback.