[22:32:59] Hi, please is possible to limit chat messages lenght in engine to some reasonable value? [22:33:28] they already are. [22:33:36] 512 chars or some such [22:33:47] Liberly land server was spammed by 9k+ message [22:34:12] update your server then? [22:34:24] when it was introduced? [22:34:28] amount of messages isn't limited [22:34:40] sapier: you sure? I could swear such a filter was added ages ago [22:34:57] wait i will bring here server owner [22:34:57] no not sure but I think only length of message is limited [22:35:24] but a mod kicking abusive users wouldn't be a big deal [22:35:29] so I don't see this to be something to be fixed in core [22:35:57] --> sol_invictus has joined this channel (~donat@pool-95-83-126-133.ptcomm.ru). [22:36:51] VanessaE: sol_invictus is liberty land owner,.. he have more informations [22:37:14] pitriss: ok. [22:39:58] ShadowNinja: ^^^^^^ ? [22:40:10] sol_invictus reports the server to be only a few days old. [22:40:28] so either that cutoff was not added or it's broken [22:43:26] well I can't remember about limiting number of messages [22:48:00] and there ain't a limitation in code [22:48:29] yet a mod doing this would be quite simple [22:50:15] http://pastebin.com/7ZWehiHZ [22:51:22] there was such a proposal at one time (hence my pinging SN. he was working on such) [22:51:29] --> Miner_48er has joined this channel (~Miner_48e@72.11.48.164). [22:51:57] working? even a full featured mod doing this was hardly more then an hour of work [22:52:28] limit in the engine, iirc [22:52:35] well it went along with some other patches he had in mind [22:52:56] so if he did start he most likely would've completet it by now [22:53:05] well we don't need this to be "in core" [22:53:09] it belongs in core. [22:53:11] why? [22:53:12] every last server owner will need it. [22:53:28] that's why. [22:53:38] oh so a server owner running a private server for his two friends and himself will need it? [22:53:39] yep ;0 [22:53:46] :) [22:53:48] for sure ;-P [22:54:27] but you know what I meant :P [22:54:29] well if he does implement it fine but it's quite ugly to do this in core while it's almost no code to be done in lua [22:55:28] http://ix.io/atr [22:55:29] there it is. [22:55:57] and some of the discussion that surrounded it: http://irc.minetest.ru/minetest-dev/2014-02-20 [22:56:07] that doesn't limit the amount of messages vanessae [22:56:09] sapier: it's not the amount that needed limited. [22:56:25] it's the length. [22:56:26] one message, 9K in length. [22:56:54] he didn't write that [22:57:00] [08-23 16:36] Hi, please is possible to limit chat messages lenght in engine to some reasonable value? [22:57:08] "chat messages length" [22:57:11] I read that as "length of a chat message" [22:57:29] not the count of messages [22:57:42] pitriss: clarify please? [22:58:20] wasn't therere already some limitation? ... ok u16 ... 65k [22:58:29] yeah [22:58:36] 65k.... is ridiculous. [22:58:39] no one's screen can hold that much [22:59:04] I really think there should be some limit in lenght.. and it should be handled by engine itself.. this is basic even in webcoding to limit user inputs to some reasonable value. [22:59:15] even my big screens could only hold about 4k or so I guess (at a readable font size) [22:59:29] pitriss: so limit the length of a single chat message? [22:59:58] yep thats good because burst of messages is not so easy to do.. [23:00:17] ok you guys discuss about a reasonable length first ;-) don't make it to short [23:00:25] and don't forget clients can still send oversized messages so this will reduce load but not fix all issues [23:00:52] burst messages are quite easy to do [23:01:30] if someone can send a 9k message he can send 90 100byte messsages too [23:02:58] but as I said fixing both of this issues in lua is possible by now too [23:03:13] <-- sol_invictus has left this server (Ping timeout: 260 seconds). [23:03:46] <-- Miner_48er has left this server (Quit: Leaving). [23:04:12] eh this guy did that by typing IMO.. there are two typos in that... [23:04:30] <-- Calinou has left this server (Ping timeout: 250 seconds). [23:04:45] typing? [23:04:52] well anyways, that ix.io link is the patch I was referring to. dunno if it's still any good. [23:06:30] discard huge messages without any action? strange behaviour who's supposed to ever find the bug why messages above 1024 don't work? [23:06:43] sapier: see the chat log. that's the same argument that was made at the time. Why didn't anyone just take the code, make it right, and push it? [23:07:04] doing this in a mod is stupid and hacky. [23:07:36] because sometime ppl refuse to clean up the mess others do ;-) .. well shadowninja does usually complete at least those parts of his work he's interested in [23:08:13] ok i will tell sol to try that patch.. i hope it will be still valid [23:08:18] but looking at current pull list he's got a lot of 90% compelte things there [23:08:30] cg72: still wanna rework the tree code? :P [23:08:56] pitriss: I already suggested via /msg that he throw together a mod to do the job for now [23:09:13] ok thank you:) [23:09:48] VanessaE yup its in progress why?