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

Control auth options #229

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Control auth options #229

wants to merge 2 commits into from

Conversation

sjaeckel
Copy link
Member

@sjaeckel sjaeckel commented Nov 1, 2023

This adds two flags to control authentication mechanisms, which is hereby put up to discussion whether it should be an opt-in or opt-out and whether a flag is appropriate for a group or we need something more flexible comparable to the priority strings of gnutls.

  • per default disable DIGEST-MD5 and PLAIN authentication, this can be enabled again by setting the flag XMPP_CONN_FLAG_WEAK_AUTH
  • allow to disable "non -PLUS" SCRAM methods via the flag XMPP_CONN_FLAG_STRONG_AUTH

@sjaeckel sjaeckel force-pushed the control-auth-options branch 3 times, most recently from d6e60a9 to 7239779 Compare November 8, 2023 12:25
@sjaeckel sjaeckel force-pushed the control-auth-options branch 4 times, most recently from 17b3587 to 941aeb2 Compare November 9, 2023 11:17
@sjaeckel sjaeckel force-pushed the scram-plus-variants branch 2 times, most recently from d6cde27 to 131dc08 Compare November 9, 2023 11:21
@sjaeckel sjaeckel force-pushed the scram-plus-variants branch 2 times, most recently from aa22dd0 to bfd0872 Compare November 27, 2023 22:50
Base automatically changed from scram-plus-variants to master January 29, 2024 09:04
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.

1 participant