Hi, On Mon, Jul 12, 2010 at 3:13 PM, Simon Gaeremynck <[email protected]> wrote: > ...emitting events for every single GET request might be a bit overkill, > that's why we only emit events if the resource has a "sling:tracking" > property set to true. > > I've created a patch for this and pushed it to codereview to get some > feedback. [2] > For now everything happens in the DefaultGetServlet as (AFAICT) most of > the requests hit that servlet first. > > WDYT, is this approach doable?...
I haven't looked at your code in detail but a gut feeling is that I would much prefer a non-intrusive approach of logging/monitoring such things. Could you maybe use a servlet Filter to generate those events? This would not require any changes to the Sling core. -Bertrand
