-
Notifications
You must be signed in to change notification settings - Fork 379
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
Allow capabilities to be handled after upstream IRC registration #1735
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1735 +/- ##
==========================================
- Coverage 39.36% 39.35% -0.01%
==========================================
Files 123 123
Lines 28198 28199 +1
Branches 99 99
==========================================
- Hits 11100 11099 -1
- Misses 17049 17051 +2
Partials 49 49
Continue to review full report at Codecov.
|
I agree |
} | ||
|
||
SendNextCap(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will send CAP END as the response to CAP NEW
One more case: 2 CAP NEW in a row, so that second one was received before CAP ACK from our REQ from the first one |
This is perhaps the first part in being able to support cap-notify and capability 3.2 by extending the capability handling to occur after registration. I would like to add some automated testing, having some problems getting those passing (debugging setup with qt/python etc) so some delays with that.
The first change removes the
m_bAuthed
check, and then the second change makes it so that subsequentCAP LS
won't trigger re-requesting already negotiated capabilities, otherwise the following would occur:Are there any other cases I am not thinking of? I am wondering about the interplay with some modules pausing capabilities after registration (perhaps PauseCap shouldn't do anything after registration?).