Re: Setting the sling.core.current.servletName request attribute

2014-01-08 Thread Dominik Süß
I'm not sure what I think of that change due to the following points: - Filters might override getAttribute and therefore could (at least theoretically) break currently working functionality (code which would not be harmful atm) - All usecases I can imagine here are about introspection of what

[jira] [Commented] (SLING-3309) Allow flushing of external caching systems

2014-01-08 Thread Marius Petria (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3309?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13865197#comment-13865197 ] Marius Petria commented on SLING-3309: -- Hi Felix, This relates to the Sling

Re: Setting the sling.core.current.servletName request attribute

2014-01-08 Thread Bertrand Delacretaz
Hi, On Tue, Jan 7, 2014 at 8:19 PM, Felix Meschberger fmesc...@adobe.com wrote: ...What about if me move setting the property to immediately before calling any filters (and resetting it after returning from all filter and script processing) ? Would that break existing functionality ?...

[replication] Whiteboard pattern for replication events?

2014-01-08 Thread Bertrand Delacretaz
Hi, This is about SLING-3309 - I'm wondering if the custom cache flushing service that Marius needs should really be in Sling, or if that should rather be a custom service that he implements outside of Sling. IIUC the scenario is as follows: 1) Changes are made to Sling content 2) The

[jira] [Commented] (SLING-3309) Allow flushing of external caching systems

2014-01-08 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3309?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13865210#comment-13865210 ] Bertrand Delacretaz commented on SLING-3309: I'm wondering if this really

Re: [replication] Whiteboard pattern for replication events?

2014-01-08 Thread Tommaso Teofili
Hi Bertrand, I think this scenario could be easily supported by Sling Replication, one could fire replication related events [1] via ReplicationEventFactory [2] once a replication request is received by a replication agent (and / or in other scenarios). The problem I see here, as far as I

[jira] [Commented] (SLING-3285) ResourceResolver.isResourceType() creates admin session for every call with super type, slowing down processing

2014-01-08 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13865235#comment-13865235 ] Konrad Windszus commented on SLING-3285: Why is creating a very short-living

Re: [replication] Whiteboard pattern for replication events?

2014-01-08 Thread Bertrand Delacretaz
Hi Tommaso, On Wed, Jan 8, 2014 at 10:17 AM, Tommaso Teofili tommaso.teof...@gmail.com wrote: ...The problem I see here, as far as I understood eventing, is related to the fact that events are not guaranteed to be consumed in the exact same order they were generated and I think this would be a

RE: [replication] Whiteboard pattern for replication events?

2014-01-08 Thread Marius Petria
Hi Bertrand, The intended patch for SLING-3309 is generic, meaning that it just allows customization of the HTTP payload (headers, body, maybe method and url?). That can be used for cache flushing but also for other things like backing up packages to a WebDav server for example. Marius

Re: [replication] Whiteboard pattern for replication events?

2014-01-08 Thread Bertrand Delacretaz
Hi Marius, On Wed, Jan 8, 2014 at 10:49 AM, Marius Petria mpet...@adobe.com wrote: ...The intended patch for SLING-3309 is generic, meaning that it just allows customization of the HTTP payload... Ok, I guess you need to show us that patch then ;-) -Bertrand

Re: [Happy new year] A new year with Sling (2013 Recap and a whishlist for 2014)

2014-01-08 Thread Dominik Süß
Hi Bertrand, On Wed, Jan 8, 2014 at 10:53 AM, Bertrand Delacretaz bdelacre...@apache.org wrote: On Wed, Jan 1, 2014 at 11:27 PM, Dominik Süß dominik.su...@gmail.com wrote: ...DevOps optimization: this includes setup, patching, monitoring and scalling of installations... I also

Re: [Happy new year] A new year with Sling (2013 Recap and a whishlist for 2014)

2014-01-08 Thread Bertrand Delacretaz
Hi Dominik, On Wed, Jan 8, 2014 at 11:34 AM, Dominik Süß dominik.su...@gmail.com wrote: ...Sling currently is based on bundles with the initial-content or installing it from the repo from install folders. In Adobe CQ Packages (vlt - contributed to Apache some months ago) are the mechanism to

Re: Setting the sling.core.current.servletName request attribute

2014-01-08 Thread Jeff Young
Hi Alex, Me thinks you didn't look hard enough. ;) Adobe CQ's WCMDebugFilter has the following (commented-out) code: /* Currently disabled because misleading information, because the script name is set after the filter chain. SlingBindings bindings = (SlingBindings)

Re: [Happy new year] A new year with Sling (2013 Recap and a whishlist for 2014)

2014-01-08 Thread Bertrand Delacretaz
Hi Dominik, On Wed, Jan 8, 2014 at 11:34 AM, Dominik Süß dominik.su...@gmail.com wrote: ...Good inline documentation is really good to understand why things behave as they behave, But you need some guidance when starting with a new technology. I think there is also some room to flatten this

Re: Setting the sling.core.current.servletName request attribute

2014-01-08 Thread Bertrand Delacretaz
On Wed, Jan 8, 2014 at 11:42 AM, Jeff Young j...@adobe.com wrote: ...FWIW, the idea came up during the investigation of a similar (but more highly-integrated) debug/developer-mode facility IMO that kind of info should come from the RequestProgressTracker instead of adding new mechanisms.

[jira] [Updated] (SLING-3309) Allow flushing of external caching systems

2014-01-08 Thread Marius Petria (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3309?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marius Petria updated SLING-3309: - Attachment: SLING-3309.patch I am adding the patch for review but it is best to apply it after

YAMF - @Source annotation

2014-01-08 Thread Bertrand Delacretaz
Hi, About https://cwiki.apache.org/confluence/display/SLING/YAMF+-+Yet+Another+Model+Factory On Tue, Jan 7, 2014 at 5:24 PM, Justin Edelson (Confluence) conflue...@apache.org wrote: // Ensure that resource is retrived from the bindings, not a request attribute @Inject

[jira] [Updated] (SLING-3308) [Javascript] Upgrading Rhino version from 1.6R2 to 1.7R4

2014-01-08 Thread rohit (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3308?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] rohit updated SLING-3308: - Attachment: (was: SLING-3308.patch) [Javascript] Upgrading Rhino version from 1.6R2 to 1.7R4

[jira] [Updated] (SLING-3308) [Javascript] Upgrading Rhino version from 1.6R2 to 1.7R4

2014-01-08 Thread rohit (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3308?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] rohit updated SLING-3308: - Attachment: SLING-3308.patch [Javascript] Upgrading Rhino version from 1.6R2 to 1.7R4

[jira] [Commented] (SLING-3308) [Javascript] Upgrading Rhino version from 1.6R2 to 1.7R4

2014-01-08 Thread rohit (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3308?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13865309#comment-13865309 ] rohit commented on SLING-3308: -- I was able to successfully run unit tests after applying null

[jira] [Resolved] (SLING-3298) PollingTransportHandler should have a configurable no. of consecutive polls

2014-01-08 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3298?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bertrand Delacretaz resolved SLING-3298. Resolution: Fixed Patch applied in revision 1556498, thanks!

[jira] [Resolved] (SLING-3299) Remote agent queue poll should be restricted as queueing agents

2014-01-08 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3299?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bertrand Delacretaz resolved SLING-3299. Resolution: Fixed Assignee: Bertrand Delacretaz Patch applied in

[jira] [Resolved] (SLING-3300) Add an API to create replication agent (configuration)

2014-01-08 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3300?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bertrand Delacretaz resolved SLING-3300. Resolution: Fixed Assignee: Bertrand Delacretaz Patch applied in revision

Re: Sling Replication patches

2014-01-08 Thread Bertrand Delacretaz
Hi Tommaso, On Tue, Jan 7, 2014 at 1:42 PM, Tommaso Teofili tommaso.teof...@gmail.com wrote: ...could anyone please have a look at the Sling Replication open Jira issues (SLING-3298, SLING-3299, SLING-3300) and eventually apply the provided patches?... Done, please cross-check. -Bertrand

Re: [replication] Whiteboard pattern for replication events?

2014-01-08 Thread Chetan Mehrotra
I think the requirement here are subset of what is required in SLING-3292[1]. So should be addressed with proper Web Hook support in Sling [1] https://issues.apache.org/jira/browse/SLING-3292 Chetan Mehrotra On Wed, Jan 8, 2014 at 3:26 PM, Bertrand Delacretaz bdelacre...@apache.org wrote: Hi

Re: Setting the sling.core.current.servletName request attribute

2014-01-08 Thread Jeff Young
Hi Bertrand, I wouldn't be opposed to using a better-structured RequestProgressTracker log. (I did originally look at the existing one, but wasn't too excited about parsing it to get the info out.) But I don't think it saves adding a new mechanism. The new mechanism's primary job is to weave

[jira] [Commented] (SLING-3300) Add an API to create replication agent (configuration)

2014-01-08 Thread Tommaso Teofili (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3300?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13865347#comment-13865347 ] Tommaso Teofili commented on SLING-3300: Thanks a lot Bertrand, it seems that the

Re: Setting the sling.core.current.servletName request attribute

2014-01-08 Thread Bertrand Delacretaz
Hi Jeff, On Wed, Jan 8, 2014 at 12:40 PM, Jeff Young j...@adobe.com wrote: ...I don't think it saves adding a new mechanism. The new mechanism's primary job is to weave the debug info into the rendered html so that it can be displayed client-side in the context of the portions of the page it

Re: Sling Replication patches

2014-01-08 Thread Tommaso Teofili
Hi Bertrand, thanks a lot, apart from a (new) class not been committed in SLING-3300 everything else looks good. Regards, Tommaso 2014/1/8 Bertrand Delacretaz bdelacre...@apache.org Hi Tommaso, On Tue, Jan 7, 2014 at 1:42 PM, Tommaso Teofili tommaso.teof...@gmail.com wrote: ...could

[jira] [Commented] (SLING-3309) Allow flushing of external caching systems

2014-01-08 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3309?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13865337#comment-13865337 ] Chetan Mehrotra commented on SLING-3309: Probably the requirement here can be met

[jira] [Commented] (SLING-3300) Add an API to create replication agent (configuration)

2014-01-08 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3300?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13865359#comment-13865359 ] Bertrand Delacretaz commented on SLING-3300: Ah sorry, missed the svn add -

[jira] [Commented] (SLING-3300) Add an API to create replication agent (configuration)

2014-01-08 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3300?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13865362#comment-13865362 ] Bertrand Delacretaz commented on SLING-3300: Small nitpick in the servlet, I

Re: YAMF - @Source annotation

2014-01-08 Thread Justin Edelson
Hi, On Wed, Jan 8, 2014 at 5:58 AM, Bertrand Delacretaz bdelacre...@apache.org wrote: Hi, About https://cwiki.apache.org/confluence/display/SLING/YAMF+-+Yet+Another+Model+Factory On Tue, Jan 7, 2014 at 5:24 PM, Justin Edelson (Confluence) conflue...@apache.org wrote: // Ensure that

Re: YAMF - @Source annotation

2014-01-08 Thread Bertrand Delacretaz
On Wed, Jan 8, 2014 at 2:52 PM, Justin Edelson jus...@justinedelson.com wrote: @Inject(source=script-bindings) ... That's not how JSR-303 was defined... Ah ok, I didn't realize you were using that - of course. -Bertrand

[jira] [Updated] (SLING-3310) Use a released parent POM in Sling Replication

2014-01-08 Thread Tommaso Teofili (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3310?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tommaso Teofili updated SLING-3310: --- Attachment: SLING-3310.patch trivial patch attached Use a released parent POM in Sling

[jira] [Created] (SLING-3310) Use a released parent POM in Sling Replication

2014-01-08 Thread Tommaso Teofili (JIRA)
Tommaso Teofili created SLING-3310: -- Summary: Use a released parent POM in Sling Replication Key: SLING-3310 URL: https://issues.apache.org/jira/browse/SLING-3310 Project: Sling Issue

Re: Setting the sling.core.current.servletName request attribute

2014-01-08 Thread Felix Meschberger
Hi I don't think pumping up the log tracker to extract information more easily is a way to go: Other than having some kind of defined line format, the actual messages are just strings with not defined format at all. So relying on the format of the messages is depending on implementation

Incorporation of HTTP Signature Authentication to facilitate automated code/configuration/topology management

2014-01-08 Thread Mark Adamcin
Hi Sling devs, I recently released a Java implementation [1] of Joyent's HTTP Signature authentication scheme [2] based on SSH authorized_keys login. The intended purpose of this draft authentication scheme is really to provide a slightly more secure alternative to Basic and Digest

Re: Incorporation of HTTP Signature Authentication to facilitate automated code/configuration/topology management

2014-01-08 Thread Justin Edelson
Hi Mark, This is very cool. I think adding this authentication handler into Sling makes sense. I'm not sure that this is the best place for the core signature code itself -- maybe that should be a separate ASF project? Don't get me wrong - I don't *mind* it being in Sling, but it seems like

Re: Incorporation of HTTP Signature Authentication to facilitate automated code/configuration/topology management

2014-01-08 Thread Mark Adamcin
Hi Justin, I'm already managing the httpsig-java code and documentation in GitHub, using Travis CI, and releasing to Maven Central under my own groupId, so I don't think there is a pressing need for that to be moved to ASF, but I'm certainly open to learning the finer details of IP management.

Re: Incorporation of HTTP Signature Authentication to facilitate automated code/configuration/topology management

2014-01-08 Thread Justin Edelson
Mark- Unless I'm missing something, clients (whether those running inside Sling or standalone) would use httpsig-java's API, not the auth handler. Correct? In any case, I think just moving the auth handler into Sling is OK, as long as the license on the httpsig-java bundle permits us to include

Re: Incorporation of HTTP Signature Authentication to facilitate automated code/configuration/topology management

2014-01-08 Thread Mark Adamcin
Yes, that's correct. And the httpsig OSGi bundle should be ASL compatible from top-to-bottom, though I would appreciate having a second look at the BouncyCastle embeds, which I identify in a NOTICE.txt file within the bundle. Mark Adamcin http://adamcin.net/ On Wed, Jan 8, 2014 at 11:14 AM,

[jira] [Commented] (SLING-3285) ResourceResolver.isResourceType() creates admin session for every call with super type, slowing down processing

2014-01-08 Thread Alexander Klimetschek (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13866005#comment-13866005 ] Alexander Klimetschek commented on SLING-3285: -- The problem seems to lie in

Re: Setting the sling.core.current.servletName request attribute

2014-01-08 Thread Alexander Klimetschek
On 08.01.2014, at 06:41, Felix Meschberger fmesc...@adobe.com wrote: I don't think pumping up the log tracker to extract information more easily is a way to go: Other than having some kind of defined line format, the actual messages are just strings with not defined format at all. So relying

Re: Sling Event 3.2.0 issues.

2014-01-08 Thread Felix Meschberger
Hi Am 08.01.2014 um 13:12 schrieb Ian Boston i...@tfd.co.uk: Hi, I am seeing a thread leak in Sling Event 3.2.0 which is fixed in the TimedEventSender in 3.3.0, however Sling Event 3.2.0 depends on later versions of Commons Scheduler which has caused some issues upgrading. Class cast

[jira] [Commented] (SLING-3285) ResourceResolver.isResourceType() creates admin session for every call with super type, slowing down processing

2014-01-08 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13866419#comment-13866419 ] Konrad Windszus commented on SLING-3285: Hi Alex, thanks for your answer but how

[jira] [Comment Edited] (SLING-3285) ResourceResolver.isResourceType() creates admin session for every call with super type, slowing down processing

2014-01-08 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13866419#comment-13866419 ] Konrad Windszus edited comment on SLING-3285 at 1/9/14 7:48 AM:

Re: Sling Event 3.2.0 issues.

2014-01-08 Thread Ian Boston
Hi, On 9 January 2014 05:57, Felix Meschberger fmesc...@adobe.com wrote: Hi Am 08.01.2014 um 13:12 schrieb Ian Boston i...@tfd.co.uk: Hi, I am seeing a thread leak in Sling Event 3.2.0 which is fixed in the TimedEventSender in 3.3.0, however Sling Event 3.2.0 depends on later versions of