trowbotham / url-parser Goto Github PK

@TRowbotham currently the parser still support PHP5 which will hit EOL at the end of the year. Do you plan on stopping PHP5 support anytime soon ?

Basically, there are 3 questions here:

1. Do you want to be notified when the parser encounters a validation error?
2. Should there be a way to enable and disable validation error reporting?
3. What would be the best way to convey the error information?

There are roughly 60 different places in the parser that currently call for a validation error to be reported. As far as I can tell, I have 3 options for reporting the error. I can use error_log(), trigger_error(), or I can push each error onto an array that can then be retrieved through a method added to the URL class. I am, of course, open to other ideas.

This change occurred in https://unicode-org.atlassian.net/browse/ICU-13311. Running input through UConverter::transcode() is no longer sufficient to transform illegal byte sequences into U+FFFD replacement characters. Unfortunately, mb_scrub() doesn't work in this case either.

This is causing a failure in the URLSearchParams constructor test on systems using ICU 60.1+.

First up: I really love this library. After a long search it was exactly what I needed.

Unfortunately, there are hosters our there that offer php 7.1+ with an outdated ICU version, which causes the usage of certain IDN constants to fail. Would you be amendable to implementing a similar fix to this one?