Skip to content

Remove OpenIdAuthorizer and OpenID authentication dependencies#17320

Merged
jt2594838 merged 2 commits intomasterfrom
codex/remove-openidauthorizer-and-deps
Mar 20, 2026
Merged

Remove OpenIdAuthorizer and OpenID authentication dependencies#17320
jt2594838 merged 2 commits intomasterfrom
codex/remove-openidauthorizer-and-deps

Conversation

@HTHou
Copy link
Contributor

@HTHou HTHou commented Mar 19, 2026

Summary

  • remove the OpenIdAuthorizer implementation and its dedicated DataNode tests
  • drop the ConfigNode login special case for OpenID users and fall back to the standard permission lookup path
  • remove OpenID-related configuration entries and JWT/OIDC dependencies from module and root Maven metadata
  • update dependencies.json and LICENSE-binary to match the reduced dependency set

Testing

  • mvn clean verify -DskipTests -P enable-sbom-check

@sonarqubecloud
Copy link

@codecov
Copy link

codecov bot commented Mar 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 39.89%. Comparing base (278b05b) to head (4c7726e).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##             master   #17320    +/-   ##
==========================================
  Coverage     39.89%   39.89%            
  Complexity      312      312            
==========================================
  Files          5117     5116     -1     
  Lines        344511   344398   -113     
  Branches      43894    43886     -8     
==========================================
- Hits         137448   137407    -41     
+ Misses       207063   206991    -72     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jt2594838 jt2594838 merged commit f7b7088 into master Mar 20, 2026
38 checks passed
@jt2594838 jt2594838 deleted the codex/remove-openidauthorizer-and-deps branch March 20, 2026 04:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants