[jira] [Updated] (SLING-4347) When a timezone is provided in a Date, it should be stored as provided

2015-01-23 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-4347?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] santiago garcía pimentel updated SLING-4347: Description: Whenever a POST request is made and it contains a date string

RE: [PROPOSAL] Context-specific configuration for Apache Sling, Multitenancy

2015-01-23 Thread Stefan Seifert
hello oliver. what's the status here? Will you move this project to Sling? this is not decided yet, no precise plans yet. this depends if it is useful for a broader audience in the sling community and projects/applications built with sling. until then just try it out from wcm.io. additionally

[jira] [Closed] (SLING-3865) Remove JcrResource.adaptTo(URL

2015-01-23 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3865?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler closed SLING-3865. --- Remove JcrResource.adaptTo(URL -- Key: SLING-3865

[jira] [Closed] (SLING-4184) Provide path mapping from JCR nodes to resource paths

2015-01-23 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4184?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler closed SLING-4184. --- Provide path mapping from JCR nodes to resource paths

[jira] [Closed] (SLING-3867) Remove node/property adaptions

2015-01-23 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3867?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler closed SLING-3867. --- Remove node/property adaptions -- Key: SLING-3867

[jira] [Closed] (SLING-4116) Content length check in JCRNodeResourceMetadata triggers javax.jcr.ItemNotFoundException: No primary item present on node

2015-01-23 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4116?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler closed SLING-4116. --- Content length check in JCRNodeResourceMetadata triggers javax.jcr.ItemNotFoundException: No

[jira] [Closed] (SLING-4307) Avoid caching JCR property values

2015-01-23 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4307?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler closed SLING-4307. --- Avoid caching JCR property values - Key:

[jira] [Created] (SLING-4347) When a timezone is provided in a Date, it should be stored as provided

2015-01-23 Thread JIRA
santiago garcía pimentel created SLING-4347: --- Summary: When a timezone is provided in a Date, it should be stored as provided Key: SLING-4347 URL: https://issues.apache.org/jira/browse/SLING-4347

[jira] [Commented] (SLING-4258) JSON representation of Calendar values should preserve timezone

2015-01-23 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-4258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289514#comment-14289514 ] santiago garcía pimentel commented on SLING-4258: - I'm creating SLING-4347

[VOTE RESULT] Release Apache Sling JCR Resource 2.4.2

2015-01-23 Thread Carsten Ziegeler
The vote passed with three binding +1 votes Carsten -- Carsten Ziegeler Adobe Research Switzerland cziege...@apache.org

[jira] [Commented] (SLING-4347) When a timezone is provided in a Date, it should be stored as provided

2015-01-23 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-4347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289515#comment-14289515 ] santiago garcía pimentel commented on SLING-4347: - [~bdelacretaz] what do

Jenkins build is back to normal : sling-trunk-1.6 #2972

2015-01-23 Thread Apache Jenkins Server
See https://builds.apache.org/job/sling-trunk-1.6/2972/changes

Jenkins build is back to normal : sling-trunk-1.7 #1360

2015-01-23 Thread Apache Jenkins Server
See https://builds.apache.org/job/sling-trunk-1.7/1360/changes

[jira] [Resolved] (SLING-4258) JSON representation of Calendar values should preserve timezone

2015-01-23 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4258?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bertrand Delacretaz resolved SLING-4258. Resolution: Fixed I have now committed your contribution in

[jira] [Updated] (SLING-4212) Sling Models: Allow multiple values from ValueMap in the resource-path injector

2015-01-23 Thread Stefan Seifert (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4212?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Stefan Seifert updated SLING-4212: -- Attachment: resourcePath-API_noRequiredCheck.patch attached is another updated version of the

Jenkins build is unstable: sling-trunk-1.8 #652

2015-01-23 Thread Apache Jenkins Server
See https://builds.apache.org/job/sling-trunk-1.8/652/changes

Jenkins build became unstable: sling-trunk-1.6 #2973

2015-01-23 Thread Apache Jenkins Server
See https://builds.apache.org/job/sling-trunk-1.6/2973/changes

[jira] [Resolved] (SLING-4212) Sling Models: Allow multiple values from ValueMap in the resource-path injector

2015-01-23 Thread Stefan Seifert (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4212?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Stefan Seifert resolved SLING-4212. --- Resolution: Fixed Completed: At revision: 1654430 patch applied Sling Models: Allow

[VOTE] Release Apache Sling JCR Resource 2.4.4

2015-01-23 Thread Carsten Ziegeler
Hi, we solved one critical issue https://issues.apache.org/jira/browse/SLING-4343 Staging repository: https://repository.apache.org/content/repositories/orgapachesling-1177/ You can use this UNIX script to download the release and verify the signatures:

Jenkins build is back to stable : sling-trunk-1.8 #653

2015-01-23 Thread Apache Jenkins Server
See https://builds.apache.org/job/sling-trunk-1.8/653/changes

[jira] [Commented] (SLING-4212) Sling Models: Allow multiple values from ValueMap in the resource-path injector

2015-01-23 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-4212?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289967#comment-14289967 ] santiago garcía pimentel commented on SLING-4212: -

Re: [VOTE] Release Apache Sling JCR Resource 2.4.4

2015-01-23 Thread Carsten Ziegeler
+1 Carsten Am 23.01.15 um 13:20 schrieb Carsten Ziegeler: Hi, we solved one critical issue https://issues.apache.org/jira/browse/SLING-4343 Staging repository: https://repository.apache.org/content/repositories/orgapachesling-1177/ You can use this UNIX script to download the release

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Radu Cotescu
On Fri, Jan 23, 2015 at 1:59 PM, Oliver Lietz apa...@oliverlietz.de wrote: Whatever the approach is, if we want to allow multiple scripting engines to use the same extension we need to refactor the code from the SlingScriptEngineManager. Why? The ScriptEngine itself can decide if it

[jira] [Updated] (SLING-4346) Sightly: HtmlParser does not parse documents correctly when comments span across internal char buffer size

2015-01-23 Thread Vlad Bailescu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4346?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vlad Bailescu updated SLING-4346: - Attachment:

[jira] [Comment Edited] (SLING-4318) Sling resource API does not expose any versioning features - bounty offered.

2015-01-23 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-4318?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289243#comment-14289243 ] Tomek Rękawek edited comment on SLING-4318 at 1/23/15 1:39 PM:

[jira] [Resolved] (SLING-4342) [Sightly] Reduce the number of repository reads

2015-01-23 Thread Felix Meschberger (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4342?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger resolved SLING-4342. -- Resolution: Fixed Assignee: Felix Meschberger Thanks for providing the patch. I

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Felix Meschberger
Hi Am 23.01.2015 um 13:29 schrieb Radu Cotescu r...@apache.org: On Fri, Jan 23, 2015 at 1:59 PM, Oliver Lietz apa...@oliverlietz.de wrote: Whatever the approach is, if we want to allow multiple scripting engines to use the same extension we need to refactor the code from the

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Felix Meschberger
Hi Am 23.01.2015 um 11:55 schrieb Oliver Lietz apa...@oliverlietz.de: On Friday 23 January 2015 10:30:26 Felix Meschberger wrote: Hi Radu Thanks for starting this thread. As I wrote in my other message [1], the ScriptEngine management is currently laid out to have script engines to

[jira] [Resolved] (SLING-4344) Modifying i18n language node in a path outside the resource resolver's search path will not invalidate the ResourceBundle cache

2015-01-23 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4344?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus resolved SLING-4344. Resolution: Fixed Fix Version/s: i18n 2.3.4 Modifying i18n language node in a path

Re: [VOTE] Release Apache Sling Launchpad Base 4.6.0-2.5.6

2015-01-23 Thread Robert Munteanu
+1 Robert On Thu, Jan 22, 2015 at 11:12 PM, Carsten Ziegeler cziege...@apache.org wrote: Hi, we solved one major deadlock issue https://issues.apache.org/jira/browse/SLING/fixforversion/12329291 Staging repository: https://repository.apache.org/content/repositories/orgapachesling-1176/

Re: SlingQuery - exported API

2015-01-23 Thread Tomek Rękawek
Hello Robert, On Thu, Jan 22, 2015 at 11:23 AM, Robert Munteanu romb...@apache.org wrote: Hi, While doing a recent build of SlingQuery I realized that all packages are exported with the project's version, so we need to add package-info.java files and start versioning. Thanks for pointing

Jenkins build is back to stable : sling-trunk-1.6 #2974

2015-01-23 Thread Apache Jenkins Server
See https://builds.apache.org/job/sling-trunk-1.6/2974/changes

[jira] [Commented] (SLING-4344) Modifying i18n language node in a path outside the resource resolver's search path will not invalidate the ResourceBundle cache

2015-01-23 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14288957#comment-14288957 ] Konrad Windszus commented on SLING-4344: Applied a slightly modified patch in rev

Re: [VOTE] Release Apache Sling Testing JCR Mock 1.1.2, ResourceResolver Mock 1.1.2, Sling Mock 1.1.2, Sling Mock Jackrabbit 0.1.2

2015-01-23 Thread Robert Munteanu
+1 Robert On Thu, Jan 22, 2015 at 11:10 PM, Stefan Seifert sseif...@pro-vision.de wrote: Hi, We solved 2 issues in JCR Mock 1.1.2: https://issues.apache.org/jira/browse/SLING/fixforversion/12329091 We solved 2 issues in ResourceResolver Mock 1.1.2:

Re: [VOTE] Release Apache Sling JCR Resource 2.4.2

2015-01-23 Thread Antonio Sanso
+1 On Jan 19, 2015, at 6:33 PM, Carsten Ziegeler cziege...@apache.org wrote: Hi, we solved some issues https://issues.apache.org/jira/browse/SLING/fixforversion/12328895 Staging repository: https://repository.apache.org/content/repositories/orgapachesling-1174/ You can use this UNIX

Re: Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Radu Cotescu
On Fri, Jan 23, 2015 at 1:15 PM, Bertrand Delacretaz bdelacre...@apache.org wrote: Good point - we can consider that notation say anywhere in the first 200 chars of the script, so ?xml version=1.0 encoding=UTF-8? !-- sling:language: thymeleaf -- foo/ would work, the pattern being

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Robert Munteanu
On Fri, Jan 23, 2015 at 1:59 PM, Oliver Lietz apa...@oliverlietz.de wrote: Whatever the approach is, if we want to allow multiple scripting engines to use the same extension we need to refactor the code from the SlingScriptEngineManager. Why? The ScriptEngine itself can decide if it kicks in

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Radu Cotescu
On Fri, Jan 23, 2015 at 2:08 PM, Robert Munteanu rob...@lmn.ro wrote: If we configure various ScriptEngine implementations it becomes harder to get an overview of what it configured where. It's even possible to have conflicting patterns in two scripting engines. We could have a master

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Oliver Lietz
On Friday 23 January 2015 13:33:37 Radu Cotescu wrote: On Fri, Jan 23, 2015 at 1:15 PM, Bertrand Delacretaz bdelacre...@apache.org wrote: Good point - we can consider that notation say anywhere in the first 200 chars of the script, so ?xml version=1.0 encoding=UTF-8? !--

[jira] [Created] (SLING-4346) Sightly: HtmlParser does not parse documents correctly when comments span across internal char buffer size

2015-01-23 Thread Vlad Bailescu (JIRA)
Vlad Bailescu created SLING-4346: Summary: Sightly: HtmlParser does not parse documents correctly when comments span across internal char buffer size Key: SLING-4346 URL:

[jira] [Created] (SLING-4348) Sync agents do not process the queue anymore

2015-01-23 Thread Marius Petria (JIRA)
Marius Petria created SLING-4348: Summary: Sync agents do not process the queue anymore Key: SLING-4348 URL: https://issues.apache.org/jira/browse/SLING-4348 Project: Sling Issue Type: Bug

[jira] [Updated] (SLING-4348) Sync agents do not process the queue anymore

2015-01-23 Thread Marius Petria (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marius Petria updated SLING-4348: - Component/s: Distribution Sync agents do not process the queue anymore

[jira] [Commented] (SLING-4348) Sync agents do not process the queue anymore

2015-01-23 Thread Marius Petria (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14290484#comment-14290484 ] Marius Petria commented on SLING-4348: -- Committed revision 1654477. Sync agents do

[jira] [Resolved] (SLING-4348) Sync agents do not process the queue anymore

2015-01-23 Thread Marius Petria (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marius Petria resolved SLING-4348. -- Resolution: Fixed Sync agents do not process the queue anymore

[jira] [Assigned] (SLING-4348) Sync agents do not process the queue anymore

2015-01-23 Thread Marius Petria (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marius Petria reassigned SLING-4348: Assignee: Marius Petria Sync agents do not process the queue anymore

Re: org.apache.sling.urlrewriter installation

2015-01-23 Thread Antonio Sanso
On Jan 22, 2015, at 6:50 PM, Oliver Lietz apa...@oliverlietz.demailto:apa...@oliverlietz.de wrote: On Thursday 22 January 2015 15:17:37 Antonio Sanso wrote: On Jan 22, 2015, at 3:52 PM, Oliver Lietz apa...@oliverlietz.demailto:apa...@oliverlietz.de wrote: On Thursday 22 January 2015 13:02:05

[jira] [Commented] (SLING-4330) provide a configuration parameter for paths patterns Sightly should handle

2015-01-23 Thread Oliver Lietz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14288996#comment-14288996 ] Oliver Lietz commented on SLING-4330: -

Re: [jira] [Commented] (SLING-4330) provide a configuration parameter for paths patterns Sightly should handle

2015-01-23 Thread Felix Meschberger
Threads collide .. lets continue the discussion on Radu’s thread [discussion] allow Script Engines to also render components based on paths“ Regards Felix Am 23.01.2015 um 11:19 schrieb Felix Meschberger fmesc...@adobe.com: Hi So we have a problem in that two script engines are using the

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Bertrand Delacretaz
On Fri, Jan 23, 2015 at 11:30 AM, Felix Meschberger fmesc...@adobe.com wrote: We could extend our ScriptResolution as well like this. If the file begins with a line of the form #!engine-nameCRLF Why not. It's a bit costly as you need to read part of the script during resolution but I

Re: Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Bertrand Delacretaz
On Fri, Jan 23, 2015 at 11:55 AM, Oliver Lietz apa...@oliverlietz.de wrote: ...Adding a Shebang to XML and HTML makes them invalid Good point - we can consider that notation say anywhere in the first 200 chars of the script, so ?xml version=1.0 encoding=UTF-8? !-- sling:language: thymeleaf

[jira] [Commented] (SLING-4330) provide a configuration parameter for paths patterns Sightly should handle

2015-01-23 Thread Radu Cotescu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289037#comment-14289037 ] Radu Cotescu commented on SLING-4330: - The {{ResourceResolution}} class is responsible

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Oliver Lietz
On Friday 23 January 2015 11:52:20 Bertrand Delacretaz wrote: On Fri, Jan 23, 2015 at 11:30 AM, Felix Meschberger fmesc...@adobe.com wrote: We could extend our ScriptResolution as well like this. If the file begins with a line of the form #!engine-nameCRLF Why not. It's a bit costly

Re: [jira] [Commented] (SLING-4330) provide a configuration parameter for paths patterns Sightly should handle

2015-01-23 Thread Felix Meschberger
Hi So we have a problem in that two script engines are using the same script/template extension ? I think we cannot handle this with a mechanism such as SLING-4330. I also think that script engines should not limit themselves in where they would like to be evaluated. That’s none of their

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Felix Meschberger
Hi Radu Thanks for starting this thread. As I wrote in my other message [1], the ScriptEngine management is currently laid out to have script engines to use disjoint extensions. Having two engines with the same extension is not supported. I also don’t really like to have multiple engines to

Re: Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Oliver Lietz
On Friday 23 January 2015 10:30:26 Felix Meschberger wrote: Hi Radu Thanks for starting this thread. As I wrote in my other message [1], the ScriptEngine management is currently laid out to have script engines to use disjoint extensions. Having two engines with the same extension is not

Re: Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Bertrand Delacretaz
On Fri, Jan 23, 2015 at 12:15 PM, Bertrand Delacretaz bdelacre...@apache.org wrote: ... the pattern being sling:language followed by whitespace and one word which is the language name Or maybe sling:script to avoid confusion with human languages. -Bertrand

[GitHub] sling pull request: SLING-4318 Servlet rendering a list of version...

2015-01-23 Thread trekawek
GitHub user trekawek opened a pull request: https://github.com/apache/sling/pull/58 SLING-4318 Servlet rendering a list of versions for the given resource. You can merge this pull request into a Git repository by running: $ git pull https://github.com/trekawek/sling

Re: sling-trunk buildbot CI build causing problems

2015-01-23 Thread Robert Munteanu
On Fri, Jan 23, 2015 at 1:03 AM, Mark Thomas ma...@apache.org wrote: Sling developers, The sling-trunk CI build managed to kill one of the buildbot slaves by filling this directory with files until the file system ran out of inodes:

[GitHub] sling pull request: SLING-4318 :restore operation for the SlingPos...

2015-01-23 Thread trekawek
GitHub user trekawek opened a pull request: https://github.com/apache/sling/pull/59 SLING-4318 :restore operation for the SlingPostServlet You can merge this pull request into a Git repository by running: $ git pull https://github.com/trekawek/sling

[jira] [Commented] (SLING-4318) Sling resource API does not expose any versioning features - bounty offered.

2015-01-23 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4318?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289229#comment-14289229 ] ASF GitHub Bot commented on SLING-4318: --- GitHub user trekawek opened a pull request:

[jira] [Commented] (SLING-4318) Sling resource API does not expose any versioning features - bounty offered.

2015-01-23 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4318?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289231#comment-14289231 ] ASF GitHub Bot commented on SLING-4318: --- GitHub user trekawek opened a pull request:

[jira] [Commented] (SLING-4212) Sling Models: Allow multiple values from ValueMap in the resource-path injector

2015-01-23 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-4212?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289232#comment-14289232 ] santiago garcía pimentel commented on SLING-4212: -

[jira] [Commented] (SLING-4258) JSON representation of Calendar values should preserve timezone

2015-01-23 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-4258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289234#comment-14289234 ] santiago garcía pimentel commented on SLING-4258: - [~bdelacretaz] do you

[jira] [Commented] (SLING-4318) Sling resource API does not expose any versioning features - bounty offered.

2015-01-23 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-4318?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289243#comment-14289243 ] Tomek Rękawek commented on SLING-4318: -- The {{;v=1.0}} parameter support has been

Re: [VOTE] Release Apache Sling Testing JCR Mock 1.1.2, ResourceResolver Mock 1.1.2, Sling Mock 1.1.2, Sling Mock Jackrabbit 0.1.2

2015-01-23 Thread Carsten Ziegeler
+1 Carsten -- Carsten Ziegeler Adobe Research Switzerland cziege...@apache.org

Re: [VOTE] Release Apache Sling Launchpad Base 4.6.0-2.5.6

2015-01-23 Thread Carsten Ziegeler
+1 Carsten -- Carsten Ziegeler Adobe Research Switzerland cziege...@apache.org

RE: [Sightly] provide access to the Script Resource Resolver in RenderContext

2015-01-23 Thread Rob Ryan
I share Felix's concern. If the ServletResolver's resolver is passed around, please ensure it is a service user with minimal permissions required for ServletResolver's needs. We do *not* want to allow a resolver passed around from the ServletResolver to be abused for accessing general

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Carsten Ziegeler
What about adding an additional property to the script resource? Carsten -- Carsten Ziegeler Adobe Research Switzerland cziege...@apache.org

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Felix Meschberger
Reminds me of the Mac Resource Forks … :-) It would be neat to be able to set this in/on the script itself without having to manage some out-of-band metadata. Regards Felix Am 23.01.2015 um 15:15 schrieb Carsten Ziegeler cziege...@apache.org: What about adding an additional property to the

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Carsten Ziegeler
Am 23.01.15 um 06:04 schrieb Felix Meschberger: Adding a Shebang to XML and HTML makes them invalid. So that won't work. Technically yes, but from a processing perspective no: because the SlingScriptAdapterFactory will it it up and the ScriptEngine should not be able to see it anymore.

[jira] [Resolved] (SLING-4346) Sightly: HtmlParser does not parse documents correctly when comments span across internal char buffer size

2015-01-23 Thread Felix Meschberger (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4346?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger resolved SLING-4346. -- Resolution: Fixed Fix Version/s: (was: Scripting Sightly Engine 1.0.2)

Re: [Sightly] provide access to the Script Resource Resolver in RenderContext

2015-01-23 Thread Felix Meschberger
Hi Am 23.01.2015 um 15:11 schrieb Rob Ryan rr...@adobe.com: I share Felix's concern. If the ServletResolver's resolver is passed around, please ensure it is a service user with minimal permissions required for ServletResolver's needs. That’s a good point: This resource resolver should

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Justin Edelson
FWIW, I think it would be more natural to set this on the container node, so if you have /apps/foo/bar/something.html, I'd set /apps/foo/bar@sling:script = html:thymeleaf. We could, however, support both per-script and per-container settings, but I don't think it is necessary to go more than one

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Chetan Mehrotra
On Fri, Jan 23, 2015 at 7:45 PM, Carsten Ziegeler cziege...@apache.org wrote: What about adding an additional property to the script resource? +1. This looks more safer and less disrupting as reading the property is cheaper. The script header approach looks nice but would require changes at

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Radu Cotescu
On Fri, Jan 23, 2015 at 4:21 PM, Chetan Mehrotra chetan.mehro...@gmail.com wrote: +1. This looks more safer and less disrupting as reading the property is cheaper. But we'd need a mixin, as scripts are usually nt:file nodes. Right?

[jira] [Commented] (SLING-4337) Sightly should only log errors from use providers if none of the use providers succeeded

2015-01-23 Thread Justin Edelson (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289353#comment-14289353 ] Justin Edelson commented on SLING-4337: --- Proposed patch:

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Bertrand Delacretaz
On Fri, Jan 23, 2015 at 3:41 PM, Chetan Mehrotra chetan.mehro...@gmail.com wrote: On Fri, Jan 23, 2015 at 7:59 PM, Radu Cotescu r...@apache.org wrote: But we'd need a mixin, as scripts are usually nt:file nodes. Right? ...Other way would be to leverage the jcr:mimeType on the child jcr:content

[jira] [Commented] (SLING-4258) JSON representation of Calendar values should preserve timezone

2015-01-23 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289328#comment-14289328 ] Bertrand Delacretaz commented on SLING-4258: I should be able to apply the

Re: sling-trunk buildbot CI build causing problems

2015-01-23 Thread Bertrand Delacretaz
Hi, On Fri, Jan 23, 2015 at 2:23 PM, Robert Munteanu romb...@apache.org wrote: ...I'm wondering if we still need the buildbot setup as the Jenkins builds seem to be (mostly) stable these days Right, I agree that we could remove the buildbot setup for now. -Bertrand

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Gabriel Walt
Hi, Isn't it possible to have some form of configuration that allows to register scripting languages based on paths when needed? E.g. by default we could following config: *.jsp = JSP engine *.html = Thymeleaf engine And then by path we could override: *.html in /foo = Sightly engine It would

[jira] [Assigned] (SLING-3440) Clarify script resolution documentation with regard to selectors

2015-01-23 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3440?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus reassigned SLING-3440: -- Assignee: Konrad Windszus Clarify script resolution documentation with regard to

[jira] [Commented] (SLING-4340) SLING-4247 created odd side-effects

2015-01-23 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4340?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289334#comment-14289334 ] Carsten Ziegeler commented on SLING-4340: - I think we should come to a conclusion

Re: sling-trunk buildbot CI build causing problems

2015-01-23 Thread Bertrand Delacretaz
Hi Mark, On Fri, Jan 23, 2015 at 12:03 AM, Mark Thomas ma...@apache.org wrote: ...However, please could you take a look at the buildbot configuration for this build and figure out a) why this happened and b) how to stop it happening again For now, feel free to disable those jobs while we

[jira] [Commented] (SLING-3440) Clarify script resolution documentation with regard to selectors

2015-01-23 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3440?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14289452#comment-14289452 ] Konrad Windszus commented on SLING-3440: I extended the documentation in r1654253.

Re: [Sightly] provide access to the Script Resource Resolver in RenderContext

2015-01-23 Thread Radu Cotescu
Hi Rob, The RuntimeExtensions are meant to process the outcome of a Sightly expression. If such an extension would need to perform some processing based on component resolution it's fair to pass it the same resolver that the ServletResolver used. I'll document in the RenderContext that this

Re: [discussion] allow Script Engines to also render components based on paths

2015-01-23 Thread Radu Cotescu
On Fri, Jan 23, 2015 at 4:41 PM, Chetan Mehrotra chetan.mehro...@gmail.com wrote: Other way would be to leverage the jcr:mimeType on the child jcr:content node which are mostly of type nt:resource. text/thymeleaf, text/sightly, text/jsp...? :)

[jira] [Comment Edited] (SLING-4342) [Sightly] Reduce the number of repository reads

2015-01-23 Thread Radu Cotescu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4342?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14287633#comment-14287633 ] Radu Cotescu edited comment on SLING-4342 at 1/23/15 12:40 PM:

[jira] [Resolved] (SLING-3440) Clarify script resolution documentation with regard to selectors

2015-01-23 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3440?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus resolved SLING-3440. Resolution: Fixed Clarify script resolution documentation with regard to selectors