Re: svn commit: r1577529 - in /sling/trunk/contrib/extensions/mongodb: pom.xml src/main/java/org/apache/sling/mongodb/impl/MongoDBCollectionResource.java src/main/java/org/apache/sling/mongodb/impl/Mo

2014-03-16 Thread Carsten Ziegeler
Good catch, thanks - it's fixed now 2014-03-15 14:30 GMT+01:00 Justin Edelson jus...@justinedelson.com: Hi, On Fri, Mar 14, 2014 at 9:49 AM, cziege...@apache.org wrote: Author: cziegeler Date: Fri Mar 14 13:49:01 2014 New Revision: 1577529 URL: http://svn.apache.org/r1577529

[VOTE RESULT] Release Apache Sling Engine 2.3.2

2014-03-16 Thread Carsten Ziegeler
The vote passes with three binding +1 votes from Mike Müller, Felix Meschberger, and Carsten Ziegeler and one non binding +1 vote from Oliver Lietz Thanks for voting Carsten -- Carsten Ziegeler cziege...@apache.org

Re: [RT] Support for deep reads from a value map

2014-03-16 Thread Carsten Ziegeler
Hi Justin it's right we can get the same result without adding this method, however with adding this method we really make clear that there is a contract change. We don't need a refactoring of the Resource api, just an extension - which imho we should have done from the beginning. Carsten

[jira] [Closed] (SLING-3439) SlingRequestProcessor.processRequest(httpServletRequest, httpServletResponse, resourceResolver) no longer passes request params through

2014-03-16 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3439?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler closed SLING-3439. --- SlingRequestProcessor.processRequest(httpServletRequest, httpServletResponse,

[jira] [Commented] (SLING-3447) Add more Source and CI links to sidebar

2014-03-16 Thread Ravi Teja (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13937100#comment-13937100 ] Ravi Teja commented on SLING-3447: -- [~rombert] Yep, make sense. Will do open up new

Build failed in Jenkins: sling-contrib-1.6 » Apache Sling JMX Resource Provider #1114

2014-03-16 Thread Apache Jenkins Server
See https://builds.apache.org/job/sling-contrib-1.6/org.apache.sling$org.apache.sling.jmx.provider/1114/changes Changes: [cziegeler] SLING-3451 : Support for deep reads from a value map [cziegeler] SLING-3450 : JMX Resource Provider should not use instanceof checks [cziegeler]

Build failed in Jenkins: sling-contrib-1.6 #1114

2014-03-16 Thread Apache Jenkins Server
See https://builds.apache.org/job/sling-contrib-1.6/1114/changes Changes: [cziegeler] SLING-3451 : Support for deep reads from a value map [olli] SLING-3017 improve KarafTestSupport, add tests and test stubs for all features [olli] SLING-3027 update and improve features [olli] SLING-3027

[jira] [Commented] (SLING-3435) ResourceAccessSecurity does not secure access for update operations

2014-03-16 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-3435?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13937288#comment-13937288 ] Mike Müller commented on SLING-3435: Ok, I reviewed to code and made some minor

[jira] [Resolved] (SLING-3435) ResourceAccessSecurity does not secure access for update operations

2014-03-16 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-3435?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mike Müller resolved SLING-3435. Resolution: Fixed Committed in r1578141 ResourceAccessSecurity does not secure access for update

[RT] Make ResourceAccessSecurity always restrict access if installed and no ResourceAccessGate present

2014-03-16 Thread Mike Müller
Hi As I worked on SLING-3435 [1] and added some more tests I noticed that Even if resourceaccesssecurity is installed as a bundle the two implementing classes ApplicationResourceAccessSecurityImpl (for application context) and ProviderResourceAccessSecurityImpl (for provider context) are only

Re: [RT] Make ResourceAccessSecurity always restrict access if installed and no ResourceAccessGate present

2014-03-16 Thread Carsten Ziegeler
Hi, the current implementation does actually this (if I read the code correctly) :) If useRAS is set, but no Gate available, the resource is not returned. At least ProviderHandler#getReadableResource does this. But of course if this is not the case, then you're totally right and we need to change