Hi, Your message has been moderated. Please subscribe to the dev ML for next exchanges
http://ofbiz.apache.org/mailing-lists.html Thanks Jacques Le 09/07/2018 à 19:43, nig...@gmail.com a écrit :
Currently, we can define request-map which invokes Java method, like following example: <request-map uri="searchAddToCategory"> <security https="true" auth="true"/> <event type="java" path="org.ofbiz.product.product.ProductSearchEvents" invoke="searchAddToCategory"/> <response name="success" type="view" value="keywordsearch"/> <response name="error" type="view" value="keywordsearch"/> </request-map> Security element just checks, if user is Signed in or not. If you want to add permission check, you will have to keep adding code inside java method. And if developer forgets, it opens up security whole. We can do security check on Views, but that is too late, in the sense code is already executed inside the method. In our code we have already added following additional attributes, which checks permissions before it executes the code. <request-map uri="searchAddToCategory"> <security https="true" auth="true" > <condition> <if-has-permission permission="XYZ" action="_CREATE"/> </condition> </security> <event type="java" path="org.ofbiz.product.product.ProductSearchEvents" invoke="searchAddToCategory"/> <response name="success" type="view" value="keywordsearch"/> <response name="error" type="view" value="keywordsearch"/> </request-map> I have patch for this, I am wondering if we like to add as part of feature or does anyone have better idea to handle.