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

coretasks, test: support CHGHOST command/capability #2116

Merged
merged 2 commits into from
Jul 3, 2021
Merged

Conversation

dgw
Copy link
Member

@dgw dgw commented Jun 24, 2021

Description

Reduces churn in bot.users and bot.channels on networks that support the chghost CAP, since they send a flood of QUIT, JOIN, and MODE lines to connected clients without chghost support in order to emulate the CHGHOST behavior.

Checks another box in #971. Also fixes a likely typo in a nearby coretasks test.

Checklist

  • I have read CONTRIBUTING.md
  • I can and do license this contribution under the EFLv2
  • No issues are reported by make qa (runs make quality and make test)
  • I have tested the functionality of the things this change touches

Notes

chghost spec for reviewers' reference.

@dgw dgw added the Feature label Jun 24, 2021
@dgw dgw added this to the 8.0.0 milestone Jun 24, 2021
@dgw dgw requested a review from a team June 24, 2021 05:40
Copy link
Contributor

@Exirel Exirel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty nice.

sopel/coretasks.py Outdated Show resolved Hide resolved
@dgw dgw requested a review from Exirel June 24, 2021 07:47
dgw added a commit to dgw/ircv3.github.io that referenced this pull request Jul 1, 2021
dgw added 2 commits July 3, 2021 00:41
Reduces churn in `bot.users` and `bot.channels` on networks that support
the `chghost` CAP, since they send a flood of QUIT, JOIN, and MODE lines
to connected clients without `chghost` support in order to emulate the
CHGHOST behavior.

Also fixes a likely typo in a nearby coretasks test.
@dgw dgw force-pushed the chghost-support branch from e270ddb to 4edb9ef Compare July 3, 2021 05:42
@dgw dgw merged commit 707a157 into master Jul 3, 2021
@dgw dgw deleted the chghost-support branch July 3, 2021 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants