Re: SlingMainServlet (almost) eats IOException, how to fix?

2015-07-30 Thread Carsten Ziegeler
I would assume that there is some subclass of IOException thrown in the cases where the client disconnects. So we could ignore just those? Carsten Am 30.07.15 um 17:58 schrieb Bertrand Delacretaz: > Hi, > > Since SLING-3498 the SlingMainServlet.service method does this: > > try { > ...do

Re: [VOTE] Release Apache Sling XSS Protection API 1.0.4

2015-07-30 Thread Daniel Klco
+1 On Thu, Jul 30, 2015 at 6:02 AM, Carsten Ziegeler wrote: > +1 > > -- > Carsten Ziegeler > Adobe Research Switzerland > cziege...@apache.org >

[jira] [Commented] (SLING-3498) Don't log stacktrace on IOException

2015-07-30 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-3498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14647865#comment-14647865 ] Bertrand Delacretaz commented on SLING-3498: The problem is that this hides le

SlingMainServlet (almost) eats IOException, how to fix?

2015-07-30 Thread Bertrand Delacretaz
Hi, Since SLING-3498 the SlingMainServlet.service method does this: try { ...do request processing } catch (IOException ioe) { log.debug("service: Probably client aborted request or any other network problem", ioe); } catch (Throwable T) { log.error(... The idea was to a

[jira] [Commented] (SLING-4919) Require configuration for webdav related services

2015-07-30 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4919?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14647844#comment-14647844 ] Carsten Ziegeler commented on SLING-4919: - I'll add them once we release and updat

[jira] [Commented] (SLING-4919) Require configuration for webdav related services

2015-07-30 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4919?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14647833#comment-14647833 ] Bertrand Delacretaz commented on SLING-4919: Won't this break integration test

[jira] [Resolved] (SLING-4919) Require configuration for webdav related services

2015-07-30 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4919?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler resolved SLING-4919. - Resolution: Fixed > Require configuration for webdav related services > --

[jira] [Created] (SLING-4919) Require configuration for webdav related services

2015-07-30 Thread Carsten Ziegeler (JIRA)
Carsten Ziegeler created SLING-4919: --- Summary: Require configuration for webdav related services Key: SLING-4919 URL: https://issues.apache.org/jira/browse/SLING-4919 Project: Sling Issue T

[jira] [Updated] (SLING-4895) Service registry should not be called from within synchronized block

2015-07-30 Thread Marius Petria (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4895?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marius Petria updated SLING-4895: - Attachment: SLING-4895.1.diff I created another patch that uses a placeholder registration object

[jira] [Commented] (SLING-4895) Service registry should not be called from within synchronized block

2015-07-30 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4895?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14647559#comment-14647559 ] Carsten Ziegeler commented on SLING-4895: - 1. Yes, that's true - this can in theor

[jira] [Commented] (SLING-4895) Service registry should not be called from within synchronized block

2015-07-30 Thread Marius Petria (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4895?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14647550#comment-14647550 ] Marius Petria commented on SLING-4895: -- I am afraid it is a bit harder than I thought

[jira] [Commented] (SLING-4895) Service registry should not be called from within synchronized block

2015-07-30 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4895?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14647535#comment-14647535 ] Carsten Ziegeler commented on SLING-4895: - [~mpetria] Yes, duplicate mappings are

[jira] [Commented] (SLING-4895) Service registry should not be called from within synchronized block

2015-07-30 Thread Marius Petria (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4895?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14647514#comment-14647514 ] Marius Petria commented on SLING-4895: -- [~cziegeler] I'll try to understand the thing

Re: Updating a release repository during a vote

2015-07-30 Thread Bertrand Delacretaz
On Thu, Jul 30, 2015 at 1:05 PM, Robert Munteanu wrote: >... I've updated the release management page Thanks! Your changes look good to me. -Bertrand

[jira] [Commented] (SLING-4895) Service registry should not be called from within synchronized block

2015-07-30 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4895?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14647477#comment-14647477 ] Carsten Ziegeler commented on SLING-4895: - [~mpetria] Yes, maybe - however I imple

Re: Updating a release repository during a vote

2015-07-30 Thread Robert Munteanu
On Mon, 2015-07-27 at 14:43 +0200, Bertrand Delacretaz wrote: > Hi, > > On Mon, Jul 20, 2015 at 3:25 PM, Robert Munteanu > wrote: > > On Mon, 2015-07-20 at 15:12 +0200, Oliver Lietz wrote: > > > ... Although versions are cheap for us, it confuses our users > > > when > > > versions are > > > "m

[jira] [Commented] (SLING-4895) Service registry should not be called from within synchronized block

2015-07-30 Thread Marius Petria (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4895?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14647433#comment-14647433 ] Marius Petria commented on SLING-4895: -- [~cziegeler] I have to look more closely to y

Re: [VOTE] Release Apache Sling Content Distribution Core version 0.1.6

2015-07-30 Thread Carsten Ziegeler
Am 30.07.15 um 09:17 schrieb Robert Munteanu: > On Wed, 2015-07-29 at 14:25 +, Marius Petria wrote: >> Please vote to approve this release: > > +1 > > Robert > +1 -- Carsten Ziegeler Adobe Research Switzerland cziege...@apache.org

Re: [VOTE] Release Apache Sling XSS Protection API 1.0.4

2015-07-30 Thread Carsten Ziegeler
+1 -- Carsten Ziegeler Adobe Research Switzerland cziege...@apache.org

Re: [jira] [Resolved] (SLING-4917) BasicInitialContentIT fails for the org.apache.sling.jcr.contentloader module

2015-07-30 Thread Carsten Ziegeler
Am 30.07.15 um 09:11 schrieb Oliver Lietz: > On Thursday 30 July 2015 08:29:24 Carsten Ziegeler wrote: >> Am 29.07.15 um 23:02 schrieb Oliver Lietz: >>> Radu, >>> >>> the issue is fixed, but I really don't like the use of the startup handler >>> from org.apache.sling.launchpad.karaf in contentloade

[jira] [Commented] (SLING-4913) Allow exclusion filters on vault package builder

2015-07-30 Thread Nicolas Peltier (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4913?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14647362#comment-14647362 ] Nicolas Peltier commented on SLING-4913: thx [~mpetria], i confirm you {Code}/home

[jira] [Resolved] (SLING-4556) NPE in DiscoveryServiceImpl#activate due to OakSlingRepositoryManager restart during startup

2015-07-30 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4556?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Munteanu resolved SLING-4556. Resolution: Fixed Fixed with https://svn.apache.org/r1693392 > NPE in DiscoveryServiceImpl#

Re: SLING-4556 - Moving org.apache.sling.installer.factory.configuration to the :boot feature

2015-07-30 Thread Robert Munteanu
On Tue, 2015-07-28 at 17:22 +0200, Bertrand Delacretaz wrote: > Hi, > > On Tue, Jul 28, 2015 at 5:15 PM, Robert Munteanu > wrote: > > ...please take a look at the patch [2] attached to SLING > > -4556 [3] and let me know if you see any issues with it... > > That looks good to me, making configu

Re: [vote] Accept the contribution of Log Tail Plugin tracked in SLING-4897

2015-07-30 Thread Robert Munteanu
On Wed, 2015-07-29 at 13:34 +0530, Varun Nagpal wrote: > Hi Robert, > > Fixed the issues related to AL headers and reattached the zip archive > with > new md5sum (69576a5b3155684659c1112eb8923274). Looks good to me, thanks! +1 for accepting the contribution Robert > > Please have a look. >

Re: [VOTE] Release Apache Sling XSS Protection API 1.0.4

2015-07-30 Thread Antonio Sanso
+1 regards antonio On Jul 29, 2015, at 5:55 PM, Radu Cotescu wrote: > Hi, > > We solved 4 issues in this release: > https://issues.apache.org/jira/browse/SLING/fixforversion/12329844 > > There are still some outstanding issues: > https://issues.apache.org/jira/browse/SLING/fixforversion/1233

Re: [VOTE] Release Apache Sling XSS Protection API 1.0.4

2015-07-30 Thread Robert Munteanu
On Wed, 2015-07-29 at 15:55 +, Radu Cotescu wrote: > Please vote to approve this release: +1 Robert

Re: [VOTE] Release Apache Sling Content Distribution Core version 0.1.6

2015-07-30 Thread Robert Munteanu
On Wed, 2015-07-29 at 14:25 +, Marius Petria wrote: > Please vote to approve this release: +1 Robert

Re: [VOTE] Release Apache Sling Content Distribution Core version 0.1.6

2015-07-30 Thread Robert Munteanu
On Thu, 2015-07-30 at 09:13 +0200, Stefan Egli wrote: > PS: just noted that this includes > org.apache.sling.distribution.sample-0.1.2 - I assume this is > intentional? Yes, Marius already sent a reply regarding this. Robert > > Cheers, > Stefan > > On 7/29/15 5:09 PM, "Stefan Egli" wrote: >

[jira] [Updated] (SLING-4818) rep:membersList nodes are not distributed correctly

2015-07-30 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4818?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Munteanu updated SLING-4818: --- Assignee: Marius Petria > rep:membersList nodes are not distributed correctly > --

[jira] [Updated] (SLING-4808) Empty config values parsed in VaultDistributionPackageBuilderFactory should be treated as null

2015-07-30 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4808?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Munteanu updated SLING-4808: --- Assignee: Marius Petria > Empty config values parsed in VaultDistributionPackageBuilderFactory

[jira] [Updated] (SLING-4834) Jcr trigger should distribute asynchronously

2015-07-30 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-4834?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Munteanu updated SLING-4834: --- Assignee: Marius Petria > Jcr trigger should distribute asynchronously >

Re: [VOTE] Release Apache Sling XSS Protection API 1.0.4

2015-07-30 Thread Stefan Egli
+1 Cheers, Stefan On 7/29/15 5:55 PM, "Radu Cotescu" wrote: >Hi, > >We solved 4 issues in this release: >https://issues.apache.org/jira/browse/SLING/fixforversion/12329844 > >There are still some outstanding issues: >https://issues.apache.org/jira/browse/SLING/fixforversion/12333112 > >Staging

Re: [VOTE] Release Apache Sling Content Distribution Core version 0.1.6

2015-07-30 Thread Stefan Egli
PS: just noted that this includes org.apache.sling.distribution.sample-0.1.2 - I assume this is intentional? Cheers, Stefan On 7/29/15 5:09 PM, "Stefan Egli" wrote: >+1 > >Cheers, >Stefan > >On 7/29/15 4:25 PM, "Marius Petria" wrote: > >>Hi, >> >> >> >>We solved 6 issues in this release: >>htt

Re: [jira] [Resolved] (SLING-4917) BasicInitialContentIT fails for the org.apache.sling.jcr.contentloader module

2015-07-30 Thread Oliver Lietz
On Thursday 30 July 2015 08:29:24 Carsten Ziegeler wrote: > Am 29.07.15 um 23:02 schrieb Oliver Lietz: > > Radu, > > > > the issue is fixed, but I really don't like the use of the startup handler > > from org.apache.sling.launchpad.karaf in contentloader as this will block > > a > > release until