https://issues.dlang.org/show_bug.cgi?id=14924

bb.temp@gmx.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bb.temp@gmx.com

--- Comment #1 from bb.temp@gmx.com ---
https://github.com/D-Programming-Language/phobos/pull/3558

By the way, the ddoc for emailLength/uriLength would need to be revised a bit, wouldn't they ?

--

https://issues.dlang.org/show_bug.cgi?id=14924

--- Comment #2 from ted <s_dlang_bugzilla@asylum.id.au> ---
Another aspect that should be noted in the documentation is that only a _very_ narrow subset of URI's are accepted....

try ftp:///foo.com (valid URI, fails 'test').....!

--

https://issues.dlang.org/show_bug.cgi?id=14924

--- Comment #3 from bb.temp@gmx.com ---
It looks like std.uri hasn't changed since it's been added in 2007. If you look at the implementation it's really old fashioned D.

For example nowadays, for `emailLength()` and `uriLength()`, the parameter
would not be of type `in char[]` but at least `string` and even perahps an
InputRange with a constraint of type  `if (isInputRange!T && isSomeChar!T)`...

I wouldn't be surprised if someone from the organisation would come and say that's it's deprecated and that instead a Regex can be used. I hang on Dlang only still one or two years so I can't say more...I may be wrong.

--

https://issues.dlang.org/show_bug.cgi?id=14924

--- Comment #4 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/169101f7bf64c2f394c39a2ba297eabd2220ecf5 fix issue 14924

https://github.com/D-Programming-Language/phobos/commit/d697e0fd4f0bcd692b0d5fedaf9baede75f8984b added unittests for issue 14924

https://github.com/D-Programming-Language/phobos/commit/3241856fe4c84f7ae4bfb075b06aa27bb878891a Merge pull request #3558 from BBasile/BBasile-patch-issue14924

fix issue14924

--

https://issues.dlang.org/show_bug.cgi?id=14924

github-bugzilla@puremagic.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--

https://issues.dlang.org/show_bug.cgi?id=14924

--- Comment #5 from github-bugzilla@puremagic.com ---
Commits pushed to stable at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/169101f7bf64c2f394c39a2ba297eabd2220ecf5 fix issue 14924

https://github.com/D-Programming-Language/phobos/commit/d697e0fd4f0bcd692b0d5fedaf9baede75f8984b added unittests for issue 14924

https://github.com/D-Programming-Language/phobos/commit/3241856fe4c84f7ae4bfb075b06aa27bb878891a Merge pull request #3558 from BBasile/BBasile-patch-issue14924

--

https://issues.dlang.org/show_bug.cgi?id=14924

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|b2.temp@gmx.com             |

--