NoCheatPlus / Issues

Issues managment for the NoCheatPlus project.
13 stars 9 forks source link

It is detected as spam a message longer about than 200 characters #341

Open leonardo-dgs opened 7 years ago

leonardo-dgs commented 7 years ago

When a player send a message even if that player are not spamming, if the message is longer than about 200 characters (in the 1.11), is detected as spam and the player receives a warning.

20.12 15:08:31 [Server] INFO ---- Version information ---- 20.12 15:08:31 [Server] INFO #### Server #### 20.12 15:08:31 [Server] INFO git-Paper-960 ~MC: 1.11~ 20.12 15:08:31 [Server] INFO detected: 1.11 20.12 15:08:31 [Server] INFO #### NoCheatPlus #### 20.12 15:08:31 [Server] INFO Plugin: 3.15.0-SNAPSHOT-sMD5NET-b1046 20.12 15:08:31 [Server] INFO MCAccess: 1.4.5-1.11|? / CB-Reflect 20.12 15:08:31 [Server] INFO Features: 20.12 15:08:31 [Server] INFO blocks: BlocksMC1_4 | BlocksMC1_5 | BlocksMC1_6_1 | BlocksMC1_7_2 | BlocksMC1_8 | BlocksMC1_9 | BlocksMC1_10 | BlocksMC1_11 20.12 15:08:31 [Server] INFO checks: FastConsume | Gutenberg | HotFixFallingBlockPortalEnter | AttackFrequency | FlyingFrequency | KeepAliveFrequency 20.12 15:08:31 [Server] INFO defaults: pvpKnockBackVelocity 20.12 15:08:31 [Server] INFO packet-listeners: UseEntityAdapter | MovingFlying | OutgoingPosition | KeepAliveAdapter | SoundDistance 20.12 15:08:31 [Server] INFO Hooks: AllViolations~NCP~ 1.0 20.12 15:08:31 [Server] INFO #### Related Plugins #### 20.12 15:08:31 [Server] INFO ProtocolLib v4.2.0-SNAPSHOT-b335

In version 1.11 send a message longer about than 200 characters without have the permission to bypass anti chat-spam and without be op.

No, I only translated the messages and disabled the motd

I run CompatNoCheatPlus

http://www.mediafire.com/view/14fplgxcpbyy8ww/2016-12-20_15.28.41.png

asofold commented 7 years ago

If the picture is just a violation log, i'd appreciate to attach the file here with GitHub directly (edit the message, look at/underneath the blue dashed line at the bottom ~ drag and drop or classic click here to upload). That'd be much better for content that's ok to stay referenced here. In some cases external sites with expiration make sense, though.

On topic: I added something to decrease the weight already. Does this happen to all long messages, or is there something specific to messages that get cancelled (e.g. lots of repetition of letters, many numbers or special characters...)?

leonardo-dgs commented 7 years ago

It's for all long messages

Platinteufel commented 7 years ago

@asofold I updated my server tonight to MC 1.11.2 and my users also getting spam warnings for long messages. The new bound should allow up to 256 characters.

I'm on the latest NoCheatPlus and the latest ProtocolLib.

asofold commented 7 years ago

@Platinteufel

Last check was that some messages got through, but maybe the capping or the entire method isn't scaling nicely (capping at 128 letters still is like the maximum length from before).

Platinteufel commented 7 years ago

@asofold I've just set chat.text.shortterm.weight from 3.0 to 2.0 and now I'm pretty fine. Changing the warning level (default 67) has no impact. Also changing chat.text.shortterm.normal has not been successful.

unbenannt

asofold commented 7 years ago

Likely the shortterm part kicks in first...

a redesign of the check would need to really make parts independent of the message length, and have message length related results processed independently (based on configuration).