You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by Robert Leland <rl...@apache.org> on 2004/04/15 04:38:05 UTC

Re: Url Validation Configuration Options

Ted Husted wrote:

>Any thoughts on this one?
>
>http://issues.apache.org/bugzilla/show_bug.cgi?id=28148
>
>"The current url validation in struts (validateUrl method in FieldChecks) only 
>uses the default url configuration options.
>
>"I am attaching a patch that will allow the following options to be set in the 
>validation.xml file using variables."
>
><snip/>
>
>-Ted.
>
You'll want to make sure it doesn't depend on:
http://issues.apache.org/bugzilla/show_bug.cgi?id=28190

It makes available long hidden functionality in Validator and it would 
be good to surface it.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


Re: Url Validation Configuration Options

Posted by Niall Pemberton <ni...@blueyonder.co.uk>.
There isn't any dependancy with Commons Bug 28190.

Bug 28190 changes commons UrlValidator to return an "error code" - if that
patch gets accepted into commons validator then we could look at changing
the Struts url validation to take advantage of this feature, but from my
position I was waiting to see if it got accepted before submitting any
changes to struts.


Niall

----- Original Message ----- 
From: "Robert Leland" <rl...@apache.org>
To: "Struts Developers List" <de...@struts.apache.org>
Sent: Thursday, April 15, 2004 3:38 AM
Subject: Re: Url Validation Configuration Options


> Ted Husted wrote:
>
> >Any thoughts on this one?
> >
> >http://issues.apache.org/bugzilla/show_bug.cgi?id=28148
> >
> >"The current url validation in struts (validateUrl method in FieldChecks)
only
> >uses the default url configuration options.
> >
> >"I am attaching a patch that will allow the following options to be set
in the
> >validation.xml file using variables."
> >
> ><snip/>
> >
> >-Ted.
> >
> You'll want to make sure it doesn't depend on:
> http://issues.apache.org/bugzilla/show_bug.cgi?id=28190
>
> It makes available long hidden functionality in Validator and it would
> be good to surface it.
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> For additional commands, e-mail: dev-help@struts.apache.org
>
>
>



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org