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.

Sign up for GitHub

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

Jump to bottom

core_commands: ctcp command #2387

Draft

slook wants to merge 2 commits into nicotine-plus:master from slook:core-commands-ctcp

Member

slook commented Feb 12, 2023

Removed: /ctcpversion command
Added: /ctcp command with query argument
Changed: Allow sending of any CTCP-quoted message, let the receiving client decide what queries it accepts and responds to
Changed: Respond with the original query within the paired response as per the CTCP protocol specification

This is backwards compatible, but obviously clients won't respond to anything other than the "VERSION" keyword.

Simpler implementation of #2381


          core_commands: ctcp command

b64845f

Member

mathiascode commented May 17, 2023

I'd like to look into CTCP after 3.3.0, but I've decided to move the /ctcpversion command to core_commands as-is for now.


          Merge branch 'master' into core-commands-ctcp

d453aff

slook marked this pull request as draft

May 19, 2023 01:28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment