commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Phil Steitz <>
Subject Re: [Validator] Localhost and email/domain validation
Date Wed, 04 May 2011 16:15:36 GMT
On 5/3/11 1:06 AM, Nick Burch wrote:
> On Mon, 4 Apr 2011, Nick Burch wrote:
>> On Mon, 28 Mar 2011, Nick Burch wrote:
>>> At the moment, the Email Validator considers @localhost email
>>> addresses to be invalid, due to the Domain Validator not
>>> considering it to be valid.
>>> Initially I thought this was wrong for all cases, so I opened
>>> VALIDATOR-292 and attached a patch that fixed this. However,
>>> after a bit more thinking and some discussions with a friend who
>>> also uses the email validator, I began to think it might need to
>>> be configurable. For a development setup, and possibly also for
>>> an intranet setup, you likely do want @localhost email addresses
>>> to be considered valid. You possibly also want to allow
>>> @machinename ones too. However, for a public facing internet app
>>> setup you likely wouldn't want that.
>>> What do people think on this? Should the Email Validator accept
>>> @localhost by default? And if not, what's the best way to allow
>>> the user to indicate that they do want that - another method (eg
>>> isValidLocal), overloading (eg isValid(String, boolean
>>> acceptLocal) or something else?
>> Any takers on this? I'm happy to re-work my patch if required,
>> but if the silence means people are happy with @localhost being
>> accepted as valid, then my existing patch could be applied as-is.
> Anyone got an opinion on which way to go with this?
> Nick

Sorry, Nick, but the repeated non-response here indicates that
[validator] is effectively dead. 

Is anyone willing to help get these patches applied and move
[validator] forward, or should we move it to dormant?

> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message