I agree with Gregg on this. If you do is as interceptor, you'll then support all the filters/extensions (including security filters)....
Ruslan -----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Gregg Bolinger Sent: June 4, 2008 12:38 PM To: Stripes Users List Subject: Re: [Stripes-users] Ann: Stripes-guicer (Guice for Stripes) It looks pretty good except it would be really nice to not have to use a special filter. An interceptor would be better with all the bells and whistles you have provided thus far. Gregg Zenin, Ruslan wrote: > It looks really good! > > I like constructor DI better too! > > So you do extend your " com.google.inject.servlet.GuiceFilter" from > Stripes filter? That means no need to configure > net.sourceforge.stripes.controller.StripesFilter? > > What about Stripes security? Are you covering it too? By Stripes > security I mean class: " > net.sourceforge.stripes.security.controller.StripesSecurityFilter" and > associated annotations...(see: > http://stripesframework.org/display/stripes/Securing+Stripes+With+ACLs) > > Regards, > Ruslan > > -----Original Message----- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] On Behalf Of Sam > Beran > Sent: June 4, 2008 11:08 AM > To: [email protected] > Subject: [Stripes-users] Ann: Stripes-guicer (Guice for Stripes) > > > Hello all, > > I've started work on a project to allow Guice DI for stripes' > ActionBeans > and ActionBeanContext. It's at http://code.google.com/p/stripes-guicer/ > http://code.google.com/p/stripes-guicer/ . I think Stripes and Guice are > a > great fit. > > I know there's been a recipe for Guice posted on this mailing list > before, > but it didn't allow constructor injection, which I prefer. If you're > interested, please check it out, and let me know if you have any > suggestions, I'm all ears! > > Sam > ------------------------------------------------------------------------ - Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php _______________________________________________ Stripes-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/stripes-users ************************************************************************ *************************** The information in this email is confidential and may be legally privileged. Access to this email by anyone other than the intended addressee is unauthorized. If you are not the intended recipient of this message, any review, disclosure, copying, distribution, retention, or any action taken or omitted to be taken in reliance on it is prohibited and may be unlawful. If you are not the intended recipient, please reply to or forward a copy of this message to the sender and delete the message, any attachments, and any copies thereof from your system. ************************************************************************ *************************** *************************************************************************************************** The information in this email is confidential and may be legally privileged. Access to this email by anyone other than the intended addressee is unauthorized. If you are not the intended recipient of this message, any review, disclosure, copying, distribution, retention, or any action taken or omitted to be taken in reliance on it is prohibited and may be unlawful. If you are not the intended recipient, please reply to or forward a copy of this message to the sender and delete the message, any attachments, and any copies thereof from your system. *************************************************************************************************** ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php _______________________________________________ Stripes-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/stripes-users
