and I think we should make it part of the existing configuration with proper prefixing in
a dedicated section.
however for the case we need to avoid case like "minLength" and stick to lower
case (except the mistake that was done for gatein.email.smtp.*).
thanks
On Feb 1, 2012, at 2:46 PM, Julien Viet wrote:
I couldn't login to JBoss yesterday my password was fucked up by
the new SSO thing.
we need to rename "username-validator.properties" to someting more generic at
least.
(otherwise we'll end up with 20 *-validator.properties" likely some day).
I will review the rest quickly today.
On Feb 1, 2012, at 2:39 PM, Christophe Laprun wrote:
> Since I haven't heard anything more on the implementation I provided yesterday, I
will consider the solution as satisfactory and go ahead with the changes needed to use the
new validator. Note that the new validator defaults to the old behavior if no user
configuration is found.
>
> On Jan 31, 2012, at 4:50 PM, Christophe Laprun wrote:
>
>>
>> On Jan 31, 2012, at 2:41 PM, Christophe Laprun wrote:
>>
>>>
>>> On Jan 31, 2012, at 2:36 PM, Julien Viet wrote:
>>>
>>>> I would like to avoid the creation of a service only for this purpose /
scope.
>>>>
>>>> can you avoid this part and make it more straightforward ?
>>>
>>> I guess UserConfigurableUsernameValidator could parse the configuration file
directly, assuming it's possible to retrieve the gatein.conf.dir value from that part
of the code.
>>
>> Simple implementation is done:
https://source.jboss.org/changelog/GateIn?cs=8330
>>
>> Now, we would just need to use it where needed. I tested that it works, the only
problem, which also occurs for ExpressionValidator, is that if the value doesn't match
the pattern, then the error message needs to output a format. The default is using the
pattern String but that doesn't work very well with HTML output…
>>
>> Cordialement / Best,
>> Chris
>>
>> ==
>> Principal Software Engineer / JBoss Enterprise Middleware Red Hat, Inc.
>> Follow GateIn:
http://blog.gatein.org /
http://twitter.com/gatein
>> Follow me:
http://metacosm.info/metacosm /
http://twitter.com/metacosm
>>
>>
>> _______________________________________________
>> gatein-dev mailing list
>> gatein-dev(a)lists.jboss.org
>>
https://lists.jboss.org/mailman/listinfo/gatein-dev
>
> Cordialement / Best,
> Chris
>
> ==
> Principal Software Engineer / JBoss Enterprise Middleware Red Hat, Inc.
> Follow GateIn:
http://blog.gatein.org /
http://twitter.com/gatein
> Follow me:
http://metacosm.info/metacosm /
http://twitter.com/metacosm
>