Bug #1487
WhoX replies not caught (or ran unnecessarily)
100%
Description
Version is actually 0.13-rc1
I've noticed on my InspIRCd 3.0 testnet (with WhoX recently implemented) that after a user joins a channel I'm in, I get WhoX replies for the other users of the channel in my Status Buffer (the WHO is sent for the user, not the channel). If I'm the joining user, I also get the same.
I added some debug logging and the WhoX replies are being processed correctly, just not being flagged as Silent. I tested a few things to no avail so far.
I'm thinking these users shouldn't even have a who query ran on them and I can't find where they are added.
I'll update this later with more specific code findings and what I've tested. Hoping someone else can provide a bit of insight.
Associated revisions
History
#1 Updated by digitalcircuit over 6 years ago
Looking into this... At this moment, the InspIRCd 3.0 testnet appears to be down, but I'll check again when it's up.
#2 Updated by digitalcircuit over 6 years ago
Fix should be in https://github.com/quassel/quassel/pull/432 ! I've put the magic "fixes" commit message to hopefully close this issue automatically once it's merged.
#3 Updated by Anonymous over 6 years ago
- Status changed from New to Resolved
- % Done changed from 0 to 100
Applied in changeset quassel|cdc6091a2e02b84a48937cda287a0769ceb8726a.
core: AutoWhoX, only search nickname by nickname
When running automatic WhoX, explicitly specify "n" to search by
nickname only. Don't rely on server defaults. This fixes an issue
with Quassel's autoWHO causing noise in the Status Buffer uncovered
by InspIRCd 3.0's implementation of WhoX that follows the spec.
Fix up incorrect attempt to escape "%" as "%%". Apparently,
QString::arg() doesn't need to escape this.
Before: WHO <nickname> %%chtsunfra,<unique_number>
After: WHO <nickname> n%chtsunfra,<unique_number>
See https://github.com/quakenet/snircd/blob/master/doc/readme.who
And https://doc.qt.io/qt-5/qstring.html#arg
Thanks to genius3000 for reporting and tracking down this issue!
Fixes #1487