I would agree with that slightly.  However with Struts 2 it is often
conveinient to have fields which are simply your Hibernate domain
models, especially if you are following paramsPrepareParams.  You might
have a form allowing them to change their name but not their signup
date.
My main concern was users setting dao/services to null or another value,
as the setters for those are required by the Spring plugin.  Is this a
valid concern, is is possible to alter these?

----- Original message -----
From: "Brian Pontarelli" <[EMAIL PROTECTED]>
To: "Struts Developers List" <dev@struts.apache.org>
Date: Thu, 25 Oct 2007 09:53:15 -0600
Subject: Re: [S2] Annotations (was Plugins gone wild!)

I tried to send a reply early, but it got rejected because it was HTML 
(oops).

I'm wondering if this is just another layer of configuration to 
alleviate flawed designs. I always find that the web tier should be 
accessible and only reveal setters for things it wants to consume. If I 
want something protected I put it in a web-business-service tier where 
it is better protected and I can control the injection and settings and 
frameworks don't introduce dangerous automatic handling.

-bp


Martin Gilday wrote:
> I've created an draft version and attached it to
> https://issues.apache.org/struts/browse/WW-2274
> It is a little bit messy at the moment and could be trimmed down a bit,
> but from my basic tests it seems to be working.
> Currently it only works on simple parameter names, so if you have a
> parameter called 'one.two' then it will not match as you only have a
> field named 'one'.  A quick way around this might be to simply trim the
> parameter name to the first '.'
>
>
> ----- Original message -----
> From: "Ted Husted" <[EMAIL PROTECTED]>
> To: "Struts Developers List" <dev@struts.apache.org>
> Date: Wed, 24 Oct 2007 14:52:16 -0400
> Subject: Re: [S2] Annotations (was Plugins gone wild!)
>
> On 10/23/07, Martin Gilday <[EMAIL PROTECTED]> wrote:
>   
>> Well I am looking at the Parameter Filter Interceptor
>> (http://cwiki.apache.org/WW/parameter-filter-interceptor.html) which I
>> am proposing we complement by allowing the same thing with annotations.
>> Currently we have a wizard like section in one of our sites which we are
>> backing with Spring session scope beans.  So the Struts2 Spring plugin
>> injects it.  To allow this we have a setMySessionBeanName(), which is
>> public.  So a user could call an action with a parameter
>> mySessionBeanName.forename and change that value.  You can stop that
>> with the filter interceptor by defining mySessionBeanName as a blocked
>> parameter name,  I would prefer to mark it @NotAParameter.
>>     
>
> Why not @blocked and @allowed for the properties, and @defaultBlock
> for the class?
>
> -Ted.
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>   


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to