ZF-6363: Tailing period is incorrectly classified as invalid
Description
Zend_Uri currently considers http://www.bob.com. invalid when in fact this is a valid uri.
Zend_Uri currently considers http://www.bob.com. invalid when in fact this is a valid uri.
Comments
Posted by Jacob Kiers (jacobkiers) on 2010-11-27T06:21:21.000+0000
According to RFC 3986 (Uniform Resource Identifier (URI): Generic Syntax), section 3.2.2 a trailing dot is perfectly valid.
A fix is created, together with a unittest. I will upload the patch as soon as my CLA is processed.
Posted by Jacob Kiers (jacobkiers) on 2010-11-30T10:42:20.000+0000
My CLA is processed and the patch is attached.
Posted by Marc Hodgins (mjh_ca) on 2010-12-09T23:28:04.000+0000
Thank you for the report and thanks for the patch Jacob. Added a unit test and applied to trunk in r23487, merged to 1.11 release branch in r23488.
Posted by Jacob Kiers (jacobkiers) on 2010-12-10T17:33:33.000+0000
Mark, I did have a test, but it seams it wasn't in the patch. I'll doublecheck next time. Thanks for creating one!
Posted by Thomas Weidner (thomas) on 2011-07-03T20:06:51.000+0000
Note that your fix does not work with some settings A fixed implementation can be found in pull request 229 for ZF2
Posted by Jacob Kiers (jacobkiers) on 2011-07-22T23:40:28.000+0000
Hi Thomas, thanks for the fix. I backported your changes and attached a patch.
Posted by Jacob Kiers (jacobkiers) on 2011-07-22T23:41:37.000+0000
Backported patch from ZF2 pull request 229 (https://github.com/zendframework/zf2/pull/229)
Posted by Jacob Kiers (jacobkiers) on 2011-07-22T23:43:38.000+0000
Re-opening issue to make it more likely someone with commit rights checks and applies the new patch.
Posted by Adam Lundrigan (adamlundrigan) on 2011-07-30T02:13:22.000+0000
Backported fix from ZF2: https://github.com/thomasweidner/zf2/…
Committed to trunk in r24306 Merged to release-1.11 in r24307
Posted by Thomas Weidner (thomas) on 2011-08-25T20:26:26.000+0000
Fixed in ZF2 with GH-279