You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by Jan Schultze <ja...@gmail.com> on 2015/06/22 10:35:51 UTC
URL validation in maven-reporting-impl
Hi,
I would like to know if there is a reason for validating URLs in
org.apache.maven.reporting.AbstractMavenReportRenderer#linkPatternedText(String)
from org.apache.maven.shared::maven-reporting-impl.
If the validation fails URLs are rendered as plain text. The class
validation uses a deprecated UrlValidator from commons-lang which considers
unqualified (i.e. local) host names and fully qualified domain names with a
top level domain longer than 4 characters as invalid (such as the special
use domain .local or more recent TLDs as .academy and .paris). This makes
using maven sites in a private network with a .local top level domain
cumbersome as browsing (proprietary) dependencies site(s) requires users to
manually copy and paste the URL.
Thanks in advance,
Jan
Re: URL validation in maven-reporting-impl
Posted by Jan Schultze <ja...@gmail.com>.
Done. See
https://issues.apache.org/jira/browse/MSHARED-429
If there is anything wrong with it, please let me know.
Hervé BOUTEMY <he...@free.fr> schrieb am Di., 23. Juni 2015 um
08:19 Uhr:
> We have a release to do on this component for other reasons (I have some
> doco
> to do...): great opportunity for this issue
>
> Did you open a Jira issue? (ideally with a patch attached :) )
>
> Regards,
>
> Hervé
>
> Le lundi 22 juin 2015 08:35:51 Jan Schultze a écrit :
> > Hi,
> >
> > I would like to know if there is a reason for validating URLs in
> >
> org.apache.maven.reporting.AbstractMavenReportRenderer#linkPatternedText(Str
> > ing) from org.apache.maven.shared::maven-reporting-impl.
> > If the validation fails URLs are rendered as plain text. The class
> > validation uses a deprecated UrlValidator from commons-lang which
> considers
> > unqualified (i.e. local) host names and fully qualified domain names
> with a
> > top level domain longer than 4 characters as invalid (such as the special
> > use domain .local or more recent TLDs as .academy and .paris). This makes
> > using maven sites in a private network with a .local top level domain
> > cumbersome as browsing (proprietary) dependencies site(s) requires users
> to
> > manually copy and paste the URL.
> >
> > Thanks in advance,
> > Jan
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>
Re: URL validation in maven-reporting-impl
Posted by Hervé BOUTEMY <he...@free.fr>.
We have a release to do on this component for other reasons (I have some doco
to do...): great opportunity for this issue
Did you open a Jira issue? (ideally with a patch attached :) )
Regards,
Hervé
Le lundi 22 juin 2015 08:35:51 Jan Schultze a écrit :
> Hi,
>
> I would like to know if there is a reason for validating URLs in
> org.apache.maven.reporting.AbstractMavenReportRenderer#linkPatternedText(Str
> ing) from org.apache.maven.shared::maven-reporting-impl.
> If the validation fails URLs are rendered as plain text. The class
> validation uses a deprecated UrlValidator from commons-lang which considers
> unqualified (i.e. local) host names and fully qualified domain names with a
> top level domain longer than 4 characters as invalid (such as the special
> use domain .local or more recent TLDs as .academy and .paris). This makes
> using maven sites in a private network with a .local top level domain
> cumbersome as browsing (proprietary) dependencies site(s) requires users to
> manually copy and paste the URL.
>
> Thanks in advance,
> Jan
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org