Skip to content

auth structs as conn meta #17557

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

Closed
wants to merge 2 commits into from
Closed

auth structs as conn meta #17557

wants to merge 2 commits into from

Conversation

icing
Copy link
Contributor

@icing icing commented Jun 8, 2025

Move auth structs to conn meta data.

Remove structs for negotiate, krb5, ntlm and gsasl from connectdata and store them as connection meta data with auto cleanup.

De-complexify sasl mech selection by moving code into static functions.

@icing icing force-pushed the conn-meta-auth branch 2 times, most recently from 96f31bd to 040c4b6 Compare June 9, 2025 12:54
Remove structs for negotiate, krb5, ntlm and gsasl from connectdata
and store them as connection meta data with auto cleanup.

De-complexify sasl mech selection by moving code into static
functions.
@icing icing marked this pull request as ready for review June 9, 2025 13:37
@icing icing requested a review from bagder June 9, 2025 13:38
@bagder bagder closed this in ab65037 Jun 10, 2025
denandz pushed a commit to denandz/curl that referenced this pull request Jun 21, 2025
Remove structs for negotiate, krb5, ntlm and gsasl from connectdata and
store them as connection meta data with auto cleanup.

De-complexify sasl mech selection by moving code into static functions.

Closes curl#17557
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants