Opened 9 years ago

Closed 9 years ago

Last modified 5 years ago

#3156 closed defect (fixed)

[PATCH] Lobby: Spam filter hides chat scrollback partially

Reported by: elexis Owned by: Josh
Priority: Should Have Milestone: Alpha 19
Component: Multiplayer lobby Keywords: easy
Cc: Patch:

Description

When connecting to the multiplayer lobby, one receives a scrollback of the last 20 or so messages.

Unfortunately it doesn't display all messages in the scrollback, since the spam check filters out the last messages.

All scrollback messages are received OnTick() and all messages are posted into the addChatMessage function in lobby.js, where they are filtered out consequently according to the rules.

It should be fixed by disabling spam check until the scrollback was received completely.

It seems the last message in the scrollback is always the room message, i.e. :

"leper has set the subject to: This is the SVN lobby, if you don't know what this means, you should install the Alpha release."

Attachments (2)

ticket3156_r16498_lobby_dont_spamfilter_chat_scrollback.patch (1.5 KB ) - added by elexis 9 years ago.
Tested - chat scrollback shows up completely and spam after that is still filtered out.
fixhistory-spam-timestamps-and-initialjoins.diff (7.9 KB ) - added by Josh 9 years ago.
Disable the spam monitor while parsing historical data, fix timestamps on historical messages, and fix superfluous initial join messages.

Download all attachments as: .zip

Change History (11)

by elexis, 9 years ago

Tested - chat scrollback shows up completely and spam after that is still filtered out.

comment:1 by elexis, 9 years ago

Keywords: review added
Summary: [Lobby] Spam filter hides chat scrollback partially[Patch] Lobby: Spam filter hides chat scrollback partially

comment:2 by elexis, 9 years ago

I think this might also be the reason why one sometimes gets instantly spamblocked for 30 seconds after rejoining the lobby.

comment:3 by Josh, 9 years ago

It would be better for the engine to add an attribute to the message containing information about what time a message was sent and whether or not it's part of the system history.

by Josh, 9 years ago

Disable the spam monitor while parsing historical data, fix timestamps on historical messages, and fix superfluous initial join messages.

comment:4 by Josh, 9 years ago

Owner: set to Josh

comment:5 by JoshuaJB, 9 years ago

Resolution: fixed
Status: newclosed

In 16507:

Fix a number of issues related to historical messages and initial join in the multiplayer lobby. Fixes #3156

comment:6 by sanderd17, 8 years ago

Keywords: review removed

comment:7 by Palaxin, 8 years ago

Summary: [Patch] Lobby: Spam filter hides chat scrollback partially[PATCH] Lobby: Spam filter hides chat scrollback partially

comment:8 by elexis, 6 years ago

In 20776:

Support both DateTime formats of XEP-0082 in the lobby, refs #3156, rP16507, rP19801, rP20032.

Patch By: Dunedan
Fixes #4920.
Differential Revision: https://code.wildfiregames.com/D1197

comment:9 by elexis, 5 years ago

In 22903:

Fix glooxwrapper::Message::when() memory leak in rP16507, refs #3156.

Differential Revision: https://code.wildfiregames.com/D2291
Tested on: clang 8.0.1

Note: See TracTickets for help on using tickets.