[GitHub] [sling-org-apache-sling-commons-scheduler] sonarcloud[bot] commented on pull request #6: SLING-11662 : added test case - ignored currently as it fails - to re…

2022-11-16 Thread GitBox
sonarcloud[bot] commented on PR #6: URL: https://github.com/apache/sling-org-apache-sling-commons-scheduler/pull/6#issuecomment-1318160824 SonarCloud Quality Gate failed. [![Quality Gate

[jira] [Commented] (SLING-11662) Endless loop in QuartzSchedulerThread.run() with maxPoolSize == queueSize

2022-11-16 Thread Carsten Ziegeler (Jira)
[ https://issues.apache.org/jira/browse/SLING-11662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17635165#comment-17635165 ] Carsten Ziegeler commented on SLING-11662: -- [~stefanegli] I updated the branch with a potential

[jira] [Comment Edited] (SLING-11458) Regress - "Writer has already been closed" exception in GraphQLServlet

2022-11-16 Thread Andreas Schaefer (Jira)
[ https://issues.apache.org/jira/browse/SLING-11458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634573#comment-17634573 ] Andreas Schaefer edited comment on SLING-11458 at 11/16/22 10:02 PM: -

[GitHub] [sling-org-apache-sling-dynamic-include] anban02 commented on pull request #26: SLING-10110: added a functionality for resolving Star-Wildcards in ig…

2022-11-16 Thread GitBox
anban02 commented on PR #26: URL: https://github.com/apache/sling-org-apache-sling-dynamic-include/pull/26#issuecomment-1317580002 I just made some adjustments to my code and trimmed it down so it is easier to read and doesn't do this hard to understand Collection-Modification. I

[GitHub] [sling-org-apache-sling-dynamic-include] sonarcloud[bot] commented on pull request #26: SLING-10110: added a functionality for resolving Star-Wildcards in ig…

2022-11-16 Thread GitBox
sonarcloud[bot] commented on PR #26: URL: https://github.com/apache/sling-org-apache-sling-dynamic-include/pull/26#issuecomment-1317564869 Kudos, SonarCloud Quality Gate passed! [![Quality Gate

[GitHub] [sling-org-apache-sling-dynamic-include] sonarcloud[bot] commented on pull request #26: SLING-10110: added a functionality for resolving Star-Wildcards in ig…

2022-11-16 Thread GitBox
sonarcloud[bot] commented on PR #26: URL: https://github.com/apache/sling-org-apache-sling-dynamic-include/pull/26#issuecomment-1317563831 Kudos, SonarCloud Quality Gate passed! [![Quality Gate

[GitHub] [sling-org-apache-sling-dynamic-include] sonarcloud[bot] commented on pull request #26: SLING-10110: added a functionality for resolving Star-Wildcards in ig…

2022-11-16 Thread GitBox
sonarcloud[bot] commented on PR #26: URL: https://github.com/apache/sling-org-apache-sling-dynamic-include/pull/26#issuecomment-1317551314 SonarCloud Quality Gate failed. [![Quality Gate

[GitHub] [sling-org-apache-sling-dynamic-include] sonarcloud[bot] commented on pull request #26: SLING-10110: added a functionality for resolving Star-Wildcards in ig…

2022-11-16 Thread GitBox
sonarcloud[bot] commented on PR #26: URL: https://github.com/apache/sling-org-apache-sling-dynamic-include/pull/26#issuecomment-1317545896 SonarCloud Quality Gate failed. [![Quality Gate

[GitHub] [sling-org-apache-sling-dynamic-include] sonarcloud[bot] commented on pull request #26: SLING-10110: added a functionality for resolving Star-Wildcards in ig…

2022-11-16 Thread GitBox
sonarcloud[bot] commented on PR #26: URL: https://github.com/apache/sling-org-apache-sling-dynamic-include/pull/26#issuecomment-1317543909 SonarCloud Quality Gate failed. [![Quality Gate

[GitHub] [sling-org-apache-sling-dynamic-include] sonarcloud[bot] commented on pull request #26: SLING-10110: added a functionality for resolving Star-Wildcards in ig…

2022-11-16 Thread GitBox
sonarcloud[bot] commented on PR #26: URL: https://github.com/apache/sling-org-apache-sling-dynamic-include/pull/26#issuecomment-1317408886 Kudos, SonarCloud Quality Gate passed! [![Quality Gate

[GitHub] [sling-org-apache-sling-testing-clients] sonarcloud[bot] commented on pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
sonarcloud[bot] commented on PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#issuecomment-1317404545 SonarCloud Quality Gate failed. [![Quality Gate

[jira] [Commented] (SLING-10110) Sling Dynamic Include: ignoreUrlParams with Wildcards do not work

2022-11-16 Thread Andreas Bannasch (Jira)
[ https://issues.apache.org/jira/browse/SLING-10110?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634922#comment-17634922 ] Andreas Bannasch commented on SLING-10110: -- [~rombert] I just created the pull-request. I'll

[GitHub] [sling-org-apache-sling-testing-clients] Kronnox commented on a diff in pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
Kronnox commented on code in PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#discussion_r1024239524 ## src/main/java/org/apache/sling/testing/Constants.java: ## @@ -37,4 +37,9 @@ public class Constants { * Http Context Attributes

[GitHub] [sling-org-apache-sling-dynamic-include] anban02 opened a new pull request, #26: SLING-10110: added a functionality for resolving Star-Wildcards in ig…

2022-11-16 Thread GitBox
anban02 opened a new pull request, #26: URL: https://github.com/apache/sling-org-apache-sling-dynamic-include/pull/26 Hello everyone about a year ago I opened the Issue [SLING-10110](https://issues.apache.org/jira/projects/SLING/issues/SLING-10110?filter=allopenissues). Now after a

[jira] [Commented] (SLING-10110) Sling Dynamic Include: ignoreUrlParams with Wildcards do not work

2022-11-16 Thread Robert Munteanu (Jira)
[ https://issues.apache.org/jira/browse/SLING-10110?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634915#comment-17634915 ] Robert Munteanu commented on SLING-10110: - [~SonglessHill] - sounds good! > Sling Dynamic

[GitHub] [sling-org-apache-sling-testing-clients] joerghoh commented on a diff in pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
joerghoh commented on code in PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#discussion_r1024218984 ## src/main/java/org/apache/sling/testing/Constants.java: ## @@ -37,4 +37,9 @@ public class Constants { * Http Context Attributes

[jira] [Comment Edited] (SLING-10110) Sling Dynamic Include: ignoreUrlParams with Wildcards do not work

2022-11-16 Thread Andreas Bannasch (Jira)
[ https://issues.apache.org/jira/browse/SLING-10110?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634899#comment-17634899 ] Andreas Bannasch edited comment on SLING-10110 at 11/16/22 4:12 PM:

[GitHub] [sling-org-apache-sling-testing-clients] bdelacretaz commented on a diff in pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
bdelacretaz commented on code in PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#discussion_r1024211896 ## src/main/java/org/apache/sling/testing/Constants.java: ## @@ -37,4 +37,9 @@ public class Constants { * Http Context Attributes

[jira] [Comment Edited] (SLING-10110) Sling Dynamic Include: ignoreUrlParams with Wildcards do not work

2022-11-16 Thread Andreas Bannasch (Jira)
[ https://issues.apache.org/jira/browse/SLING-10110?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634899#comment-17634899 ] Andreas Bannasch edited comment on SLING-10110 at 11/16/22 4:00 PM:

[jira] [Comment Edited] (SLING-10110) Sling Dynamic Include: ignoreUrlParams with Wildcards do not work

2022-11-16 Thread Andreas Bannasch (Jira)
[ https://issues.apache.org/jira/browse/SLING-10110?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634899#comment-17634899 ] Andreas Bannasch edited comment on SLING-10110 at 11/16/22 3:50 PM:

[jira] [Commented] (SLING-10110) Sling Dynamic Include: ignoreUrlParams with Wildcards do not work

2022-11-16 Thread Andreas Bannasch (Jira)
[ https://issues.apache.org/jira/browse/SLING-10110?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634899#comment-17634899 ] Andreas Bannasch commented on SLING-10110: -- I tried to add the functionality myself and wrote

[GitHub] [sling-org-apache-sling-testing-clients] Kronnox commented on a diff in pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
Kronnox commented on code in PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#discussion_r1024175976 ## src/main/java/org/apache/sling/testing/Constants.java: ## @@ -37,4 +37,9 @@ public class Constants { * Http Context Attributes

[GitHub] [sling-org-apache-sling-testing-clients] Kronnox commented on a diff in pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
Kronnox commented on code in PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#discussion_r1024158765 ## src/test/java/org/apache/sling/testing/CustomUserAgentInterceptorTest.java: ## @@ -0,0 +1,125 @@ +/* + * Licensed to the Apache Software

[GitHub] [sling-org-apache-sling-testing-clients] bdelacretaz commented on a diff in pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
bdelacretaz commented on code in PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#discussion_r1024142834 ## src/main/java/org/apache/sling/testing/Constants.java: ## @@ -37,4 +37,9 @@ public class Constants { * Http Context Attributes

[GitHub] [sling-org-apache-sling-testing-clients] sonarcloud[bot] commented on pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
sonarcloud[bot] commented on PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#issuecomment-1317177157 SonarCloud Quality Gate failed. [![Quality Gate

[GitHub] [sling-org-apache-sling-testing-clients] bdelacretaz commented on pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
bdelacretaz commented on PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#issuecomment-1317151894 Thanks for this! I've made a few minor comments but it generally looks good to me. -- This is an automated message from the Apache Git Service. To

[GitHub] [sling-org-apache-sling-testing-clients] bdelacretaz commented on a diff in pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
bdelacretaz commented on code in PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#discussion_r1024113483 ## src/test/java/org/apache/sling/testing/CustomUserAgentInterceptorTest.java: ## @@ -0,0 +1,125 @@ +/* + * Licensed to the Apache

[GitHub] [sling-org-apache-sling-testing-clients] bdelacretaz commented on a diff in pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
bdelacretaz commented on code in PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#discussion_r1024111918 ## src/main/java/org/apache/sling/testing/clients/util/UserAgent.java: ## @@ -0,0 +1,122 @@ +/* + * Licensed to the Apache Software

[GitHub] [sling-org-apache-sling-testing-clients] joerghoh commented on a diff in pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
joerghoh commented on code in PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#discussion_r1024110803 ## src/main/java/org/apache/sling/testing/Constants.java: ## @@ -37,4 +37,9 @@ public class Constants { * Http Context Attributes

[GitHub] [sling-org-apache-sling-testing-clients] bdelacretaz commented on a diff in pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
bdelacretaz commented on code in PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#discussion_r1024107734 ## src/main/java/org/apache/sling/testing/Constants.java: ## @@ -37,4 +37,9 @@ public class Constants { * Http Context Attributes

[jira] [Comment Edited] (SLING-11678) Protect the Tooling Support Install servlet

2022-11-16 Thread Carsten Ziegeler (Jira)
[ https://issues.apache.org/jira/browse/SLING-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634849#comment-17634849 ] Carsten Ziegeler edited comment on SLING-11678 at 11/16/22 2:10 PM:

[jira] [Commented] (SLING-11678) Protect the Tooling Support Install servlet

2022-11-16 Thread Carsten Ziegeler (Jira)
[ https://issues.apache.org/jira/browse/SLING-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634849#comment-17634849 ] Carsten Ziegeler commented on SLING-11678: -- times change :) There are two things: deploying a

[jira] [Commented] (SLING-11678) Protect the Tooling Support Install servlet

2022-11-16 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634842#comment-17634842 ] Konrad Windszus commented on SLING-11678: - To quote from

[GitHub] [sling-org-apache-sling-testing-clients] sonarcloud[bot] commented on pull request #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
sonarcloud[bot] commented on PR #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41#issuecomment-1316948817 SonarCloud Quality Gate failed. [![Quality Gate

[GitHub] [sling-org-apache-sling-testing-clients] Kronnox opened a new pull request, #41: SLING-XXX: Set more descriptive User-Agent for SlingClients

2022-11-16 Thread GitBox
Kronnox opened a new pull request, #41: URL: https://github.com/apache/sling-org-apache-sling-testing-clients/pull/41 Instead of the current User-Agent `JAVA` that is used on all SlingClient requests a more descriptive one (`sling-testing-client/`) is used instead. Furthermore

[jira] [Commented] (SLING-11678) Protect the Tooling Support Install servlet

2022-11-16 Thread Carsten Ziegeler (Jira)
[ https://issues.apache.org/jira/browse/SLING-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634796#comment-17634796 ] Carsten Ziegeler commented on SLING-11678: -- Agreed, looks like we could really drop this

[jira] [Commented] (SLING-11678) Protect the Tooling Support Install servlet

2022-11-16 Thread Robert Munteanu (Jira)
[ https://issues.apache.org/jira/browse/SLING-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634789#comment-17634789 ] Robert Munteanu commented on SLING-11678: - If we use the Webconsole REST API to install the

[jira] [Commented] (SLING-11678) Protect the Tooling Support Install servlet

2022-11-16 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634780#comment-17634780 ] Konrad Windszus commented on SLING-11678: - For me this functionality should have less dependency

[jira] [Commented] (SLING-11678) Protect the Tooling Support Install servlet

2022-11-16 Thread Carsten Ziegeler (Jira)
[ https://issues.apache.org/jira/browse/SLING-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634762#comment-17634762 ] Carsten Ziegeler commented on SLING-11678: -- The answer is easy, back then no OSGi Http

[jira] [Commented] (SLING-11678) Protect the Tooling Support Install servlet

2022-11-16 Thread Robert Munteanu (Jira)
[ https://issues.apache.org/jira/browse/SLING-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634758#comment-17634758 ] Robert Munteanu commented on SLING-11678: - [~kwin] - I think [~cziegeler] would know best as he

[jira] [Commented] (SLING-11678) Protect the Tooling Support Install servlet

2022-11-16 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17634716#comment-17634716 ] Konrad Windszus commented on SLING-11678: - [~rombert] WDYT? Any particular reason why you

[jira] [Created] (SLING-11678) Protect the Tooling Support Install servlet

2022-11-16 Thread Konrad Windszus (Jira)
Konrad Windszus created SLING-11678: --- Summary: Protect the Tooling Support Install servlet Key: SLING-11678 URL: https://issues.apache.org/jira/browse/SLING-11678 Project: Sling Issue