[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-25 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r1004641669 ## src/test/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunctionTest.java: ## @@ -46,9 +46,11 @@ public class ExtensionFunctionTest extends JXPathTestCase {

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-25 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r1004641669 ## src/test/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunctionTest.java: ## @@ -46,9 +46,11 @@ public class ExtensionFunctionTest extends JXPathTestCase {

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-24 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r1002987643 ## src/test/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunctionTest.java: ## @@ -46,9 +46,11 @@ public class ExtensionFunctionTest extends JXPathTestCase {

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-24 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r1002987643 ## src/test/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunctionTest.java: ## @@ -46,9 +46,11 @@ public class ExtensionFunctionTest extends JXPathTestCase {

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-24 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r1002987643 ## src/test/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunctionTest.java: ## @@ -46,9 +46,11 @@ public class ExtensionFunctionTest extends JXPathTestCase {

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-24 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r1003034601 ## src/main/java/org/apache/commons/jxpath/functions/MethodFunction.java: ## @@ -34,6 +36,8 @@ public class MethodFunction implements Function { private final Me

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-24 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r1003005419 ## src/main/java/org/apache/commons/jxpath/ri/JXPathFilter.java: ## @@ -0,0 +1,39 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * co

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-24 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r1002983191 ## src/main/java/org/apache/commons/jxpath/functions/MethodFunction.java: ## @@ -34,6 +36,8 @@ public class MethodFunction implements Function { private final Me

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-19 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r999896659 ## src/main/java/org/apache/commons/jxpath/ClassFunctions.java: ## @@ -91,6 +92,13 @@ public Function getFunction( final String namespace, final Stri

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-19 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r27379 ## src/main/java/org/apache/commons/jxpath/ClassFunctions.java: ## @@ -91,6 +92,13 @@ public Function getFunction( final String namespace, final Stri

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-19 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r999896659 ## src/main/java/org/apache/commons/jxpath/ClassFunctions.java: ## @@ -91,6 +92,13 @@ public Function getFunction( final String namespace, final Stri

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-19 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r27379 ## src/main/java/org/apache/commons/jxpath/ClassFunctions.java: ## @@ -91,6 +92,13 @@ public Function getFunction( final String namespace, final Stri

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-19 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r999896659 ## src/main/java/org/apache/commons/jxpath/ClassFunctions.java: ## @@ -91,6 +92,13 @@ public Function getFunction( final String namespace, final Stri

[GitHub] [commons-jxpath] Warxim commented on a diff in pull request #26: Fix CVE-2022-41852

2022-10-19 Thread GitBox
Warxim commented on code in PR #26: URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r999879987 ## src/main/java/org/apache/commons/jxpath/ri/JXPathFilter.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * con