Skip to content

[TASK-367] SASL/PLAIN AUTH#375

Merged
luoyuxia merged 4 commits intoapache:mainfrom
fresh-borzoni:sasl-plain-auth
Feb 28, 2026
Merged

[TASK-367] SASL/PLAIN AUTH#375
luoyuxia merged 4 commits intoapache:mainfrom
fresh-borzoni:sasl-plain-auth

Conversation

@fresh-borzoni
Copy link
Contributor

@fresh-borzoni fresh-borzoni commented Feb 25, 2026

Summary

closes #367 #374

Add SASL/PLAIN authentication support to the Rust client, matching the Java Fluss client behavior.

  • Core: Authenticate request/response, exponential backoff with jitter (matching Java's ExponentialBackoff), retriable auth error handling
  • Config: security_protocol, security_sasl_mechanism, security_sasl_username, security_sasl_password, connect_timeout_ms
  • Bindings: C++ and Python bindings expose all new config fields
  • Tests: SASL integration tests (valid creds, wrong password, unknown user) for Rust, C++ and Python
  • Infra: Consolidated Rust IT from per-module clusters into a single shared LazyLock cluster with dual listeners, enabling concurrent test execution
  • Docs: Updated configuration and error handling pages for all three languages

@fresh-borzoni
Copy link
Contributor Author

@luoyuxia @leekeiabstraction PTAL 🙏

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 30 out of 30 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@fresh-borzoni
Copy link
Contributor Author

fresh-borzoni commented Feb 25, 2026

@luoyuxia fixed issue with cpp tests, caused by additional \ used by formatter.
Would you mind to run CI/CD pipeline again to check?

Copy link
Contributor

@leekeiabstraction leekeiabstraction left a comment

Choose a reason for hiding this comment

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

TY for the PR for such an important feature! Left some comments.

@fresh-borzoni
Copy link
Contributor Author

@leekeiabstraction Ty, addressed comments, PTAL 🙏

@fresh-borzoni
Copy link
Contributor Author

@luoyuxia @leekeiabstraction Rebased and fixed conflicts. PTAL

Copy link
Contributor

@leekeiabstraction leekeiabstraction left a comment

Choose a reason for hiding this comment

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

TY for the PR!

@fresh-borzoni
Copy link
Contributor Author

rebased

Copy link
Contributor

@luoyuxia luoyuxia left a comment

Choose a reason for hiding this comment

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

@fresh-borzoni Thanks. LGTM. Merging..

@luoyuxia luoyuxia merged commit 5f8130f into apache:main Feb 28, 2026
9 checks passed
@luoyuxia
Copy link
Contributor

@fresh-borzoni
Copy link
Contributor Author

fresh-borzoni commented Feb 28, 2026

@fresh-borzoni Seems ci fails after merge this pr. https://github.com/apache/fluss-rust/actions/runs/22520032930/job/65243381417

Looking into this
UPD
#392

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.

SASL/PLAIN for client auth

4 participants