[GitHub] [sling-org-apache-sling-caconfig-integration-tests] sonarcloud[bot] commented on pull request #1: SLING-11114 update SLING API to 2.21.0

2022-06-02 Thread GitBox
sonarcloud[bot] commented on PR #1: URL: https://github.com/apache/sling-org-apache-sling-caconfig-integration-tests/pull/1#issuecomment-1145133899 Kudos, SonarCloud Quality Gate passed! [![Quality Gate

[GitHub] [sling-org-apache-sling-resourceresolver] sonarcloud[bot] commented on pull request #72: SLING-11372: resource resolver: cleanup code related to aliases

2022-06-02 Thread GitBox
sonarcloud[bot] commented on PR #72: URL: https://github.com/apache/sling-org-apache-sling-resourceresolver/pull/72#issuecomment-1145092345 SonarCloud Quality Gate failed. [![Quality Gate

[GitHub] [sling-org-apache-sling-testing-clients] sonarcloud[bot] commented on pull request #35: SLING-11364 more precise exceptions

2022-06-02 Thread GitBox
sonarcloud[bot] commented on PR #35: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/35#issuecomment-1145085430 SonarCloud Quality Gate failed. [![Quality Gate

[GitHub] [sling-org-apache-sling-testing-clients] joerghoh opened a new pull request, #35: SLING-11364 more precise exceptions

2022-06-02 Thread GitBox
joerghoh opened a new pull request, #35: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/35 use more specialized exceptions than the generic ``ClientException``. With this it should be possible that the TestDriver can give more accurate indications about the type

[GitHub] [sling-org-apache-sling-resourceresolver] sonarcloud[bot] commented on pull request #72: SLING-11372: resource resolver: cleanup code related to aliases

2022-06-02 Thread GitBox
sonarcloud[bot] commented on PR #72: URL: https://github.com/apache/sling-org-apache-sling-resourceresolver/pull/72#issuecomment-1145015198 SonarCloud Quality Gate failed. [![Quality Gate

[jira] [Commented] (SLING-11372) resource resolver: cleanup code related to aliases

2022-06-02 Thread Julian Reschke (Jira)
[ https://issues.apache.org/jira/browse/SLING-11372?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17545521#comment-17545521 ] Julian Reschke commented on SLING-11372:

[GitHub] [sling-org-apache-sling-testing-clients] joerghoh closed pull request #34: SLING-11364 use specialized exceptions

2022-06-02 Thread GitBox
joerghoh closed pull request #34: SLING-11364 use specialized exceptions URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/34 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[jira] [Created] (SLING-11372) resource resolver: cleanup code related to aliases

2022-06-02 Thread Julian Reschke (Jira)
Julian Reschke created SLING-11372: -- Summary: resource resolver: cleanup code related to aliases Key: SLING-11372 URL: https://issues.apache.org/jira/browse/SLING-11372 Project: Sling Issue

[jira] [Created] (SLING-11371) Wrong 503-unavailable status code in Distribution API responses

2022-06-02 Thread Jira
José Correia created SLING-11371: Summary: Wrong 503-unavailable status code in Distribution API responses Key: SLING-11371 URL: https://issues.apache.org/jira/browse/SLING-11371 Project: Sling

[GitHub] [sling-org-apache-sling-distribution-core] jose-correia commented on a diff in pull request #59: SLING-11366: Provide more exception error context on the JSON API responses

2022-06-02 Thread GitBox
jose-correia commented on code in PR #59: URL: https://github.com/apache/sling-org-apache-sling-distribution-core/pull/59#discussion_r887976156 ## src/main/java/org/apache/sling/distribution/servlet/DistributionAgentServlet.java: ## @@ -71,7 +73,7 @@ protected void

Re: Custom Maven Enforcer Rule hosted at Sling

2022-06-02 Thread Robert Munteanu
Sounds good to me. You could go with the other one as well, I was just curious :-) Thanks, Robert On Thu, 2022-06-02 at 14:05 +0200, Konrad Windszus wrote: > I see, then “sling-maven-enforcer-rules” and "maven-enforcer-rules” > make more sense. > WDYT? > Konrad > > > On 2. Jun 2022, at 13:58,

Re: Custom Maven Enforcer Rule hosted at Sling

2022-06-02 Thread Konrad Windszus
I see, then “sling-maven-enforcer-rules” and "maven-enforcer-rules” make more sense. WDYT? Konrad > On 2. Jun 2022, at 13:58, Robert Munteanu wrote: > > On Thu, 2022-06-02 at 09:56 +, Konrad Windszus wrote: >> Coming back to this topic after a long time I propose to use the >> repository

Re: Custom Maven Enforcer Rule hosted at Sling

2022-06-02 Thread Robert Munteanu
On Thu, 2022-06-02 at 09:56 +, Konrad Windszus wrote: > Coming back to this topic after a long time I propose to use the > repository name "sling-org-apache-sling-maven-enforcer-rules" for > this and give it artifact id "org.apache.sling.maven.enforcer.rules". > > If I don't hear any

[GitHub] [sling-org-apache-sling-resource-inventory] rombert commented on a diff in pull request #1: SLING-11370: Sling update to 47

2022-06-02 Thread GitBox
rombert commented on code in PR #1: URL: https://github.com/apache/sling-org-apache-sling-resource-inventory/pull/1#discussion_r887868976 ## bnd.bnd: ## @@ -0,0 +1 @@ +Private-Package: org.apache.sling.resource.inventory.impl Review Comment: This might not be needed, can

[jira] [Commented] (SLING-11370) Update parent bundle (47) to sling-resource-inventory

2022-06-02 Thread Ashok Pelluru (Jira)
[ https://issues.apache.org/jira/browse/SLING-11370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17545436#comment-17545436 ] Ashok Pelluru commented on SLING-11370: ---

[jira] [Created] (SLING-11370) Update parent bundle (47) to sling-resource-inventory

2022-06-02 Thread Ashok Pelluru (Jira)
Ashok Pelluru created SLING-11370: - Summary: Update parent bundle (47) to sling-resource-inventory Key: SLING-11370 URL: https://issues.apache.org/jira/browse/SLING-11370 Project: Sling

[jira] [Resolved] (SLING-11368) Update parent bundle (47) to sling-mom

2022-06-02 Thread Robert Munteanu (Jira)
[ https://issues.apache.org/jira/browse/SLING-11368?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Munteanu resolved SLING-11368. - Assignee: Robert Munteanu Resolution: Fixed PR applied, thanks

[GitHub] [sling-org-apache-sling-mom] rombert merged pull request #2: SLING-11368: Upgrade to sling 47

2022-06-02 Thread GitBox
rombert merged PR #2: URL: https://github.com/apache/sling-org-apache-sling-mom/pull/2 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [sling-org-apache-sling-mom] rombert commented on a diff in pull request #2: SLING-11368: Upgrade to sling 47

2022-06-02 Thread GitBox
rombert commented on code in PR #2: URL: https://github.com/apache/sling-org-apache-sling-mom/pull/2#discussion_r887857033 ## src/main/java/org/apache/sling/mom/package-info.java: ## @@ -20,5 +20,7 @@ /** */ -@org.osgi.annotation.versioning.Version("1.0.0")

[GitHub] [sling-org-apache-sling-mom] ashokmca07 commented on a diff in pull request #2: SLING-11368: Upgrade to sling 47

2022-06-02 Thread GitBox
ashokmca07 commented on code in PR #2: URL: https://github.com/apache/sling-org-apache-sling-mom/pull/2#discussion_r887853002 ## src/main/java/org/apache/sling/mom/package-info.java: ## @@ -20,5 +20,7 @@ /** */ -@org.osgi.annotation.versioning.Version("1.0.0")

[GitHub] [sling-org-apache-sling-mom] rombert commented on a diff in pull request #2: SLING-11368: Upgrade to sling 47

2022-06-02 Thread GitBox
rombert commented on code in PR #2: URL: https://github.com/apache/sling-org-apache-sling-mom/pull/2#discussion_r887839810 ## src/main/java/org/apache/sling/mom/package-info.java: ## @@ -20,5 +20,7 @@ /** */ -@org.osgi.annotation.versioning.Version("1.0.0")

[jira] [Updated] (SLING-11369) Provide Maven Enforcer rule which checks that transitive provided dependencies are contained in the runtime Maven classpath

2022-06-02 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-11369?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus updated SLING-11369: Description: All dependencies with provided scope are not transitively inherited. While

[jira] [Updated] (SLING-11369) Provide Maven Enforcer rule which checks that transitive provided dependencies are contained in the runtime Maven classpath

2022-06-02 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-11369?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus updated SLING-11369: Summary: Provide Maven Enforcer rule which checks that transitive provided dependencies

[jira] [Created] (SLING-11369) Provide Maven Enforcer which checks that transitive provided dependencies are contained in the runtime Maven classpath

2022-06-02 Thread Konrad Windszus (Jira)
Konrad Windszus created SLING-11369: --- Summary: Provide Maven Enforcer which checks that transitive provided dependencies are contained in the runtime Maven classpath Key: SLING-11369 URL:

Re: Custom Maven Enforcer Rule hosted at Sling

2022-06-02 Thread Konrad Windszus
Coming back to this topic after a long time I propose to use the repository name "sling-org-apache-sling-maven-enforcer-rules" for this and give it artifact id "org.apache.sling.maven.enforcer.rules". If I don't hear any objections I would create the repo at the end of the week. Konrad On

[jira] [Commented] (SLING-11367) Potential NPE as ResolveContext.getProviderState() is nullable

2022-06-02 Thread Angela Schreiber (Jira)
[ https://issues.apache.org/jira/browse/SLING-11367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17545355#comment-17545355 ] Angela Schreiber commented on SLING-11367: -- [~cziegeler], i don't think my fix would make the

[jira] [Commented] (SLING-11368) Update parent bundle (47) to sling-mom

2022-06-02 Thread Ashok Pelluru (Jira)
[ https://issues.apache.org/jira/browse/SLING-11368?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17545353#comment-17545353 ] Ashok Pelluru commented on SLING-11368: ---

[jira] [Created] (SLING-11368) Update parent bundle (47) to sling-mom

2022-06-02 Thread Ashok Pelluru (Jira)
Ashok Pelluru created SLING-11368: - Summary: Update parent bundle (47) to sling-mom Key: SLING-11368 URL: https://issues.apache.org/jira/browse/SLING-11368 Project: Sling Issue Type:

[jira] [Commented] (SLING-11367) Potential NPE as ResolveContext.getProviderState() is nullable

2022-06-02 Thread Carsten Ziegeler (Jira)
[ https://issues.apache.org/jira/browse/SLING-11367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17545338#comment-17545338 ] Carsten Ziegeler commented on SLING-11367: -- This might be a matter of taste and how it will be

Improve JSON serialization and response data types

2022-06-02 Thread José Correia
Hello! While working on the Distribution API I noticed some code smells that I would like to discuss and see if we have a solution already implemented in any Sling repository: - *Java/JSON serialization* Currently, we are implementing our own serializer in the form of a generic utils class:

[GitHub] [sling-org-apache-sling-distribution-core] jose-correia commented on a diff in pull request #59: SLING-11366: Provide more exception error context on the JSON API responses

2022-06-02 Thread GitBox
jose-correia commented on code in PR #59: URL: https://github.com/apache/sling-org-apache-sling-distribution-core/pull/59#discussion_r887647886 ## src/main/java/org/apache/sling/distribution/servlet/DistributionAgentServlet.java: ## @@ -71,7 +73,7 @@ protected void

[GitHub] [sling-org-apache-sling-distribution-core] jose-correia commented on a diff in pull request #59: SLING-11366: Provide more exception error context on the JSON API responses

2022-06-02 Thread GitBox
jose-correia commented on code in PR #59: URL: https://github.com/apache/sling-org-apache-sling-distribution-core/pull/59#discussion_r887647886 ## src/main/java/org/apache/sling/distribution/servlet/DistributionAgentServlet.java: ## @@ -71,7 +73,7 @@ protected void

[GitHub] [sling-org-apache-sling-distribution-core] jose-correia commented on a diff in pull request #59: SLING-11366: Provide more exception error context on the JSON API responses

2022-06-02 Thread GitBox
jose-correia commented on code in PR #59: URL: https://github.com/apache/sling-org-apache-sling-distribution-core/pull/59#discussion_r887647886 ## src/main/java/org/apache/sling/distribution/servlet/DistributionAgentServlet.java: ## @@ -71,7 +73,7 @@ protected void

[GitHub] [sling-org-apache-sling-distribution-core] jose-correia commented on a diff in pull request #59: SLING-11366: Provide more exception error context on the JSON API responses

2022-06-02 Thread GitBox
jose-correia commented on code in PR #59: URL: https://github.com/apache/sling-org-apache-sling-distribution-core/pull/59#discussion_r887647886 ## src/main/java/org/apache/sling/distribution/servlet/DistributionAgentServlet.java: ## @@ -71,7 +73,7 @@ protected void

[GitHub] [sling-org-apache-sling-distribution-core] tmaret commented on a diff in pull request #59: SLING-11366: Provide more exception error context on the JSON API responses

2022-06-02 Thread GitBox
tmaret commented on code in PR #59: URL: https://github.com/apache/sling-org-apache-sling-distribution-core/pull/59#discussion_r887645261 ## src/main/java/org/apache/sling/distribution/servlet/DistributionAgentServlet.java: ## @@ -71,7 +73,7 @@ protected void

[GitHub] [sling-org-apache-sling-distribution-core] jose-correia commented on a diff in pull request #59: SLING-11366: Provide more exception error context on the JSON API responses

2022-06-02 Thread GitBox
jose-correia commented on code in PR #59: URL: https://github.com/apache/sling-org-apache-sling-distribution-core/pull/59#discussion_r887636626 ## src/main/java/org/apache/sling/distribution/servlet/DistributionAgentServlet.java: ## @@ -71,7 +73,7 @@ protected void

[GitHub] [sling-org-apache-sling-distribution-core] jose-correia commented on a diff in pull request #59: SLING-11366: Provide more exception error context on the JSON API responses

2022-06-02 Thread GitBox
jose-correia commented on code in PR #59: URL: https://github.com/apache/sling-org-apache-sling-distribution-core/pull/59#discussion_r887636626 ## src/main/java/org/apache/sling/distribution/servlet/DistributionAgentServlet.java: ## @@ -71,7 +73,7 @@ protected void

[GitHub] [sling-org-apache-sling-distribution-core] jose-correia commented on a diff in pull request #59: SLING-11366: Provide more exception error context on the JSON API responses

2022-06-02 Thread GitBox
jose-correia commented on code in PR #59: URL: https://github.com/apache/sling-org-apache-sling-distribution-core/pull/59#discussion_r887636626 ## src/main/java/org/apache/sling/distribution/servlet/DistributionAgentServlet.java: ## @@ -71,7 +73,7 @@ protected void

[GitHub] [sling-org-apache-sling-resource-inventory] sonarcloud[bot] commented on pull request #1: Sling update to 47

2022-06-02 Thread GitBox
sonarcloud[bot] commented on PR #1: URL: https://github.com/apache/sling-org-apache-sling-resource-inventory/pull/1#issuecomment-1144502714 Kudos, SonarCloud Quality Gate passed! [![Quality Gate

[jira] [Comment Edited] (SLING-11175) Update to Bnd 6.3

2022-06-02 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-11175?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17500550#comment-17500550 ] Konrad Windszus edited comment on SLING-11175 at 6/2/22 6:41 AM: - Fixed

[jira] [Resolved] (SLING-11175) Update to Bnd 6.3

2022-06-02 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-11175?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus resolved SLING-11175. - Resolution: Fixed > Update to Bnd 6.3 > - > > Key:

[jira] [Updated] (SLING-11175) Update to Bnd 6.3

2022-06-02 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-11175?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus updated SLING-11175: Description: https://github.com/bndtools/bnd/wiki/Changes-in-6.3.0 (was:

[jira] [Reopened] (SLING-11175) Update to Bnd 6.2

2022-06-02 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-11175?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus reopened SLING-11175: - > Update to Bnd 6.2 > - > > Key: SLING-11175 >

[jira] [Updated] (SLING-11175) Update to Bnd 6.3

2022-06-02 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-11175?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus updated SLING-11175: Summary: Update to Bnd 6.3 (was: Update to Bnd 6.2) > Update to Bnd 6.3 >