[ 
https://issues.apache.org/jira/browse/HBASE-29557?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Nihal Jain updated HBASE-29557:
-------------------------------
    Component/s: jetty12
                 dependencies
                 REST
                 security
                 Thrift
                 UI

> Decouple dependency on Hadoop AuthenticationFilter classes
> ----------------------------------------------------------
>
>                 Key: HBASE-29557
>                 URL: https://issues.apache.org/jira/browse/HBASE-29557
>             Project: HBase
>          Issue Type: Improvement
>          Components: dependencies, jetty12, REST, security, Thrift, UI
>            Reporter: Nihal Jain
>            Assignee: Nihal Jain
>            Priority: Major
>
> Introduce a new module, hbase-auth-filters, that contains a minimal, copy of 
> the Hadoop auth pieces HBase uses (e.g., AuthenticationFilter and required 
> helpers). This decouples HBase’s server stack (web UI, REST, Thrift) from 
> Hadoop’s javax-based auth so we can move to Jetty 12 EE10/Jakarta without 
> being gated by Hadoop’s servlet/Jetty timeline.
> *Motivation*
>  - Jetty 9 is EOL; we completed the EE8 step with HBASE-29224 and want to 
> move to EE10 (Jakarta) with HBASE-29542
>  - Today our servers depend on Hadoop’s javax AuthenticationFilter. That ties 
> our upgrade pace to Hadoop’s servlet/Jetty cadence.
>  - By carrying a copy, we unblock HBase now and avoid future coordination 
> bottlenecks.
> *Proposed Change*
>  - Add a new module: hbase-auth-filters
>  - Copy only the minimal Hadoop auth classes HBase uses (AuthenticationFilter 
> + small set of helpers).
>  - Wire HBase web/REST/Thrift servers to use these classes and ensure we do 
> not have any regression as we simply copy here.
> *Pros*
>  - Fully decouples HBase’s server stack from Hadoop’s servlet pace; clean 
> Jakarta path for HBASE-29542.
>  - Avoids being perpetually tied to Hadoop’s Jetty/servlet cadence.
>  - Interoperability is preserved with:
>  -- Current Hadoop lines that remain on javax, and
>  -- Future Hadoop lines that move to Jakarta
> *Compatibility/Support Notes*
>  - Server-internal change only; no wire or client API changes expected.
>  - Allows HBase to support Hadoop versions on javax today and those on 
> Jakarta in the future without forcing a drop of javax-era Hadoop immediately 
> when Hadoop switches.
>  - If/when Hadoop publishes Jakarta-native auth, we can evaluate switching to 
> their artifacts; because we’re decoupled, that can be done on our schedule.
> *Cons/Risks*
>  - We must track upstream fixes in the copied classes.
>  - Mitigation: keep the surface area minimal; periodically review for 
> security/bug fixes. These classes have historically been relatively stable.
> *Alternatives (not chosen here)*
>  - Shade/relocate Hadoop auth into a private package and rewrite 
> javax→jakarta (hadoop-auth-shaded): avoids a source fork but retains coupling 
> to Hadoop releases and requires re-shading on upstream changes.
>  - Wait for Hadoop to move to Jakarta: simplest short-term, but keeps HBase 
> blocked on Hadoop’s schedule.
> *Acceptance Criteria*
>  - HBase servers start and operate on Jetty 12 EE8 using hbase-auth-filters 
> as it were before
>  - REST/Thrift/authn flows pass existing test suites (including Kerberos).
>  - No regression in supported Hadoop versions due to this change.
> *Follow-ups*
>  - Switch to jakarta for the our (then) native auth filter and unblock 
> HBASE-29542 without waiting for hadoop.
> *Fix Version(s)*
>  - Target: master, branch-3
>  - Although, This can be backported to branch-2 as well, paving way for our 
> future jetty migration path
> *Class list identified for hbase-auth-filters*
> Below is a minimal list of files we may have to copy from hadoop; a PoC will 
> follow if others think this approach is worth investing our time in.
> {{grep -r "javax.servlet." src/main | cut -d: -f1 | sort | uniq}}
>  * 
> src/main/java/org/apache/hadoop/security/authentication/server/AltKerberosAuthenticationHandler.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/server/AuthenticationFilter.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/server/AuthenticationHandler.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/server/AuthenticationToken.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/server/JWTRedirectAuthenticationHandler.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/server/KerberosAuthenticationHandler.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/server/LdapAuthenticationHandler.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/server/MultiSchemeAuthenticationHandler.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/server/PseudoAuthenticationHandler.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/util/CertificateUtil.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/util/FileSignerSecretProvider.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/util/RolloverSignerSecretProvider.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/util/SignerSecretProvider.java
>  * 
> src/main/java/org/apache/hadoop/security/authentication/util/ZKSignerSecretProvider.java



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to