This is an automated email from the ASF dual-hosted git repository. bonampak pushed a commit to branch feature/jakarta-jetty-upgrade in repository https://gitbox.apache.org/repos/asf/knox.git
commit 85579aaabea25cb4b97da00134a6ac327be45606 Author: bonampak <[email protected]> AuthorDate: Sun Apr 26 19:28:56 2026 +0200 KNOX-3238: temporarily make HadoopAuthFilter compile --- gateway-provider-security-hadoopauth/pom.xml | 7 +++++-- .../knox/gateway/hadoopauth/filter/HadoopAuthFilter.java | 14 +++++++------- 2 files changed, 12 insertions(+), 9 deletions(-) diff --git a/gateway-provider-security-hadoopauth/pom.xml b/gateway-provider-security-hadoopauth/pom.xml index ae417864c..76c0908e8 100755 --- a/gateway-provider-security-hadoopauth/pom.xml +++ b/gateway-provider-security-hadoopauth/pom.xml @@ -77,6 +77,11 @@ <groupId>org.apache.hadoop</groupId> <artifactId>hadoop-common</artifactId> </dependency> + <dependency> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> + <version>3.1.0</version> + </dependency> <dependency> <groupId>de.thetaphi</groupId> @@ -102,8 +107,6 @@ <scope>test</scope> </dependency> - - <dependency> <groupId>org.apache.knox</groupId> <artifactId>gateway-provider-security-pac4j</artifactId> diff --git a/gateway-provider-security-hadoopauth/src/main/java/org/apache/knox/gateway/hadoopauth/filter/HadoopAuthFilter.java b/gateway-provider-security-hadoopauth/src/main/java/org/apache/knox/gateway/hadoopauth/filter/HadoopAuthFilter.java index 0a46b5f62..0b6a14b50 100755 --- a/gateway-provider-security-hadoopauth/src/main/java/org/apache/knox/gateway/hadoopauth/filter/HadoopAuthFilter.java +++ b/gateway-provider-security-hadoopauth/src/main/java/org/apache/knox/gateway/hadoopauth/filter/HadoopAuthFilter.java @@ -104,7 +104,7 @@ public class HadoopAuthFilter extends private Set<String> unAuthenticatedPaths = new HashSet<>(20); private String topologyName; - @Override + //@Override protected Properties getConfiguration(String configPrefix, FilterConfig filterConfig) throws ServletException { GatewayServices services = GatewayServer.getGatewayServices(); AliasService aliasService = services.getService(ServiceType.ALIAS_SERVICE); @@ -112,7 +112,7 @@ public class HadoopAuthFilter extends return getConfiguration(aliasService, configPrefix, filterConfig); } - @Override + //@Override public void init(FilterConfig filterConfig) throws ServletException { this.topologyName = (String) filterConfig.getInitParameter("clusterName"); final List<String> initParameterNames = AuthFilterUtils.getInitParameterNamesAsList(filterConfig); @@ -144,7 +144,7 @@ public class HadoopAuthFilter extends ignoreDoAs.addAll(ignoredServices); } - super.init(filterConfig); + //super.init(filterConfig); final String supportJwt = filterConfig.getInitParameter(SUPPORT_JWT); final boolean jwtSupported = Boolean.parseBoolean(supportJwt == null ? "false" : supportJwt); @@ -160,7 +160,7 @@ public class HadoopAuthFilter extends AuthFilterUtils.addUnauthPaths(unAuthenticatedPaths, unAuthPathString, DEFAULT_AUTH_UNAUTHENTICATED_PATHS_PARAM); } - @Override + //@Override public void doFilter(ServletRequest request, ServletResponse response, FilterChain filterChain) throws IOException, ServletException { /* check for unauthenticated paths to bypass */ @@ -172,11 +172,11 @@ public class HadoopAuthFilter extends LOG.useJwtFilter(); jwtFilter.doFilter(request, response, filterChain); } else { - super.doFilter(request, response, filterChain); + //super.doFilter(request, response, filterChain); } } - @Override + //@Override protected void doFilter(FilterChain filterChain, HttpServletRequest request, HttpServletResponse response) throws IOException, ServletException { /* check for unauthenticated paths to bypass */ if(AuthFilterUtils.doesRequestContainUnauthPath(unAuthenticatedPaths, request)) { @@ -218,7 +218,7 @@ public class HadoopAuthFilter extends } } - super.doFilter(filterChain, proxyRequest == null ? request : proxyRequest, response); + //super.doFilter(filterChain, proxyRequest == null ? request : proxyRequest, response); } /**
