RE: patch for configuring event extension

Just a quick comment here...I agree with Thelmo that we need to make it
easier to remap the event extension, but I agree with Jake that we need a
slightly different approach than what's been proposed. I glanced at the
Barracuda code and think there are a couple of ways we could handle this,
and I'll respond more fully later today...

Christian

----------------------------------------------
Christian Cryder
Internet Architect, ATMReports.com
Project Chair, BarracudaMVC - http://barracudamvc.org
----------------------------------------------
"Coffee? I could quit anytime, just not today"


> -----Original Message-----
> From: [email protected]
> [mailto:[email protected]]On Behalf Of Jacob Kjome
> Sent: Monday, January 12, 2004 10:32 AM
> To: [email protected]
> Subject: Re: [Barracuda] patch for configuring event extension
>
>
> Quoting Thelmo Loisio <[email protected]>:
>
> > With this small patch you can configure the event extension from a
> > servlet init parameter.
> > It seems to me very easy and straightforward, i think that people in a
> > situation like mine could benefit from the flexibility of Barracuda even
> > more.
> >
> > Attached are unified diffs against current CVS.
> >
> > --
> > Thelmo
>
>
> This is a nice idea, but I'm concerned that it collides with the
> <servlet-mapping> configuration such as...
>
>     <servlet-mapping>
>         <servlet-name>ApplicationGateway</servlet-name>
>         <url-pattern>*.event</url-pattern>
>     </servlet-mapping>
>
> If I have the above and then have...
>
>     <init-param>
>         <param-name>EventExtension</param-name>
>         <param-value>.foo</param-value>
>     </init-param>
>
> What is the proper event extension; ".foo" or ".event"?  We
> really need to fall
> in line with the servlet spec here.  Let's not create our own
> configuration
> mechnanisms.  I don't have time at the moment to think about this
> much more, so
> I will discuss this more later.  The one thing I know is that
> changing the even
> extension should happen differently than what this patch proposes
> (Even though I
> realize that this isn't really changing the way Barracuda has
> always done this,
> it just providing a configuration option for it...but I think it
> needs to be
> discouraged in support of a more standard way to doing things).
>
> Jake
> _______________________________________________
> Barracuda mailing list
> [email protected]
> http://barracudamvc.org/lists/listinfo/barracuda