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

Store capability values from CAP v3.2 handshakes + honor SASL v3.2 mechanism lists #322

Merged
merged 4 commits into from
Dec 13, 2022

Conversation

progval
Copy link
Contributor

@progval progval commented Feb 14, 2022

  1. Store capability values in 'CAP LS 302' negotiations.

    So it can be retrieved later (eg. to know what SASL mechanisms are available).

    https://ircv3.net/specs/extensions/capability-negotiation.html#the-cap-ls-subcommand

  2. Honor SASL v3.2 mechanism lists

    From https://ircv3.net/specs/extensions/sasl-3.2#usage:

    Clients SHOULD pick a mechanism present in the CAP LS reply they get from the server and attempt to use that mechanism for authentication after they request the sasl capability.

Tested on The Lounge.

@ItsOnlyBinary
Copy link
Contributor

As this is tracking available caps it seems it would be also wise for it to track cap-notify (CAP NEW|DEL) events to keep the list accurate

@progval
Copy link
Contributor Author

progval commented Sep 4, 2022

Done (but not tested)

progval and others added 4 commits December 11, 2022 15:31
From https://ircv3.net/specs/extensions/sasl-3.2#usage :

> Clients SHOULD pick a mechanism present in the CAP LS reply they get from the server and attempt to use that mechanism for authentication after they request the sasl capability.
@ItsOnlyBinary
Copy link
Contributor

forcing checks to re-run

@ItsOnlyBinary ItsOnlyBinary reopened this Dec 13, 2022
@ItsOnlyBinary ItsOnlyBinary merged commit a2697c9 into kiwiirc:master Dec 13, 2022
@ItsOnlyBinary
Copy link
Contributor

:) thanks

@progval progval deleted the sasl-3.2 branch May 29, 2023 07:23
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