Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

♻️ Use SASL::ClientAdapter #194

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nevans
Copy link
Collaborator

@nevans nevans commented Oct 9, 2023

Use the experimental adapter created by #183.

@nevans nevans added the SASL 🔒 Authentication and authentication mechanisms label Oct 10, 2023
@nevans nevans force-pushed the sasl/use-client-adapter branch 3 times, most recently from bc2cc17 to 3ba0ccd Compare October 27, 2023 02:32
@nevans nevans force-pushed the sasl/use-client-adapter branch 3 times, most recently from a523d11 to 74fb75f Compare November 4, 2023 05:22
@nevans nevans added the v0.5 label May 19, 2024
@nevans nevans added this to the v0.5 milestone May 19, 2024
@nevans nevans removed the v0.5 label May 19, 2024
@nevans nevans force-pushed the sasl/use-client-adapter branch 2 times, most recently from d3d2335 to c063ac0 Compare June 16, 2024 17:44
@nevans nevans force-pushed the sasl/use-client-adapter branch 6 times, most recently from 5c633a4 to 2ff4010 Compare June 30, 2024 20:20
@nevans nevans marked this pull request as draft June 30, 2024 20:21
@nevans nevans force-pushed the sasl/use-client-adapter branch 3 times, most recently from 2e2c88e to 56b028d Compare July 7, 2024 17:07
@nevans
Copy link
Collaborator Author

nevans commented Sep 16, 2024

I'd been delaying this because I wanted to handle cancelation properly. But that isn't necessary: cancelation is no worse in the new implementation than it's ever been. So I split that work off into #324.

@nevans nevans marked this pull request as ready for review September 16, 2024 03:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SASL 🔒 Authentication and authentication mechanisms
Development

Successfully merging this pull request may close these issues.

1 participant