Skip to content

HDDS-15168. Support configurable SASL mechanism#10212

Open
adoroszlai wants to merge 6 commits intoapache:masterfrom
adoroszlai:HDDS-15168
Open

HDDS-15168. Support configurable SASL mechanism#10212
adoroszlai wants to merge 6 commits intoapache:masterfrom
adoroszlai:HDDS-15168

Conversation

@adoroszlai
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Port changes from Hadoop to the forked RPC in Ozone:

  • 78987a71a64 HADOOP-19151. Support configurable SASL mechanism
  • 9dad697dbca HADOOP-19227. ipc.Server accelerate token negotiation only for the default mechanism
  • 317db31a9a5 HADOOP-19306. Support user defined auth Callback in SaslRpcServer
  • cd2cffe73f9 HADOOP-19342. SaslRpcServer.AuthMethod print INFO messages in client side
  • 390724364d1 HADOOP-19359. Accelerate token negotiation for other similar mechanisms

https://issues.apache.org/jira/browse/HDDS-15168

How was this patch tested?

$ java -cp $(echo hadoop-ozone/dist/target/ozone-2.2.0-SNAPSHOT/share/ozone/lib/*jar | sed 's/ /:/g'):hadoop-ozone/dist/target/ozone-2.2.0-SNAPSHOT/etc/hadoop \
    org.apache.hadoop.security_.SaslMechanismFactory
SASL_MECHANISM = DIGEST-MD5

$ export HADOOP_SASL_MECHANISM=asdf-qwerty
$ java -cp $(echo hadoop-ozone/dist/target/ozone-2.2.0-SNAPSHOT/share/ozone/lib/*jar | sed 's/ /:/g'):hadoop-ozone/dist/target/ozone-2.2.0-SNAPSHOT/etc/hadoop \
    org.apache.hadoop.security_.SaslMechanismFactory
SASL_MECHANISM = asdf-qwerty

CI:
https://github.com/adoroszlai/ozone/actions/runs/25507663750

@adoroszlai adoroszlai self-assigned this May 7, 2026
@adoroszlai adoroszlai requested a review from szetszwo May 7, 2026 19:39
Copy link
Copy Markdown
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 the change looks good.

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