<div>
<div><div>Hi all,</div><div> </div><div>+1</div><div><br></div><div>I think it's useful to allow customization of the constraint. As said Shane</div><div>the annotation should provide an enum allowing to use all built-in JSoup </div><div>whitelist and allow the user to specify its whitelist implementation.</div><div><br></div><div>--Kevin</div></div><div><span><br></span>
<p style="color: #a0a0a0;">Le dimanche 12 juin 2011 à 13:14, Gunnar Morling a écrit :</p>
<blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;">
<span><div><div>Hi George,<br><br>I added some comments to your pull request.<br><br>With respect to Shane's whitelist suggestion you might have a value()<br>attribute accepting an enumeration with the standard whitelist<br>implementations and alternative attribute Class<? extends Whitelist><br>whiteListClass() which allows to specify custom whitelist classes.<br><br>--Gunnar<br><br><br>2011/6/12 George Gastaldi <<a href="mailto:gegastaldi@gmail.com">gegastaldi@gmail.com</a>>:<br><blockquote type="cite"><div>Cool ideas. Will implement it ASAP<br><br>Em 12/06/2011, às 02:06, Shane Bryzak <<a href="mailto:sbryzak@redhat.com">sbryzak@redhat.com</a>> escreveu:<br><br><blockquote type="cite"><div>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:<br><br>// Default would use Whitelist.relaxed()<br>private @WebSafe String content;<br><br>// This would use the basic whitelist<br>private @WebSafe(basic) String content;<br><br>// Basic whitelist plus images<br>private @WebSafe(basicWithImages) String content;<br><br>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.<br><br><br>On 12/06/11 12:46, George Gastaldi wrote:<br><blockquote type="cite"><div>Hey Shane,<br><br>I implemented the @WebSafe annotation you mentioned on Hibenate Validator.<br>The pull request is <a href="https://github.com/hibernate/hibernate-validator/pull/61">https://github.com/hibernate/hibernate-validator/pull/61</a><br><br>Regards,<br><br>George Gastaldi<br></div></blockquote></div></blockquote><br>_______________________________________________<br>seam-dev mailing list<br><a href="mailto:seam-dev@lists.jboss.org">seam-dev@lists.jboss.org</a><br><a href="https://lists.jboss.org/mailman/listinfo/seam-dev">https://lists.jboss.org/mailman/listinfo/seam-dev</a><br></div></blockquote></div></div></span>
</blockquote>
<div>
<br>
</div>
</div>
</div>