[seam-dev] @WebSafe annotation

[Gunnar Morling]

Hi George,

I added some comments to your pull request.

With respect to Shane's whitelist suggestion you might have a value()
attribute accepting an enumeration with the standard whitelist
implementations and alternative attribute Class<? extends Whitelist>
whiteListClass() which allows to specify custom whitelist classes.

--Gunnar


2011/6/12 George Gastaldi <gegastaldi at gmail.com>:
> Cool ideas. Will implement it ASAP
>
> Em 12/06/2011, às 02:06, Shane Bryzak <sbryzak at redhat.com> escreveu:
>
>> Looks good, although I wouldn't hard code Whitelist.basic() as the default... instead I would make Whitelist.relaxed() the default, and then perhaps allow the developer to override it by specifying an enum value for the validation level allowed on the @WebSafe annotation.  E.g:
>>
>> // Default would use Whitelist.relaxed()
>> private @WebSafe String content;
>>
>> // This would use the basic whitelist
>> private @WebSafe(basic) String content;
>>
>> // Basic whitelist plus images
>> private @WebSafe(basicWithImages) String content;
>>
>> It would even be nice to somehow allow the developer to specify their own whitelist, if you can work out an elegant way to implement it.
>>
>>
>> On 12/06/11 12:46, George Gastaldi wrote:
>>> Hey Shane,
>>>
>>> I implemented the @WebSafe annotation you mentioned on Hibenate Validator.
>>> The pull request is https://github.com/hibernate/hibernate-validator/pull/61
>>>
>>> Regards,
>>>
>>> George Gastaldi
>>
>
> _______________________________________________
> seam-dev mailing list
> seam-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/seam-dev
>