Minetest logo

IRC log for #minetest-dev, 2018-01-20

| Channels | #minetest-dev index | Today | | Google Search | Plaintext

All times shown according to UTC.

Time Nick Message
00:27 Icedream joined #minetest-dev
00:33 Sockbat joined #minetest-dev
01:46 Sockbat joined #minetest-dev
02:05 Sockbat joined #minetest-dev
02:20 Sockbat joined #minetest-dev
04:28 ssieb joined #minetest-dev
05:06 YuGiOhJCJ joined #minetest-dev
07:29 nerzhul joined #minetest-dev
07:59 red-001 nerzhul, since you're here what's your view on #6932?
07:59 ShadowBot https://github.com/minetest/minetest/issues/6932 -- Remove CSM screenshot API by red-001
08:09 nerzhul red-001, i prefer implementing a rate limit and a client setting
08:09 nerzhul if user wants to use a mod to automagically generate screens to show its life on game why not
08:10 Fritigern joined #minetest-dev
08:10 red-001 it's just that no-one wrote rate-limiting for it and even with rate limiting I would prefer if it required user interaction to enable
08:10 nerzhul yes,
08:10 red-001 kinda like the way browsers ask the users before letting websites use certain features
08:10 nerzhul rate limit + client setting by default to false
08:11 red-001 s/users/user
08:14 red-001 nerzhul, how hard would it be to isolate mods from each other in thier own lua_state and do we even want to do that?
08:15 nerzhul it's not difficult except for the callback functions
08:16 nerzhul it needs to be handled client side
08:16 nerzhul instead of lua side
08:17 red-001 is it only me or does travis take a lot longer to build stuff now?
08:18 red-001 updated 6067 yesterday and travis is still waiting with the xcode build
08:19 nerzhul macosx build is long to start since the begininng of the week and some builds randomly failed
08:19 nerzhul maybe it's due to the CPU patching on AWS cloud, some performance dropdown cna happen too
08:20 red-001 maybe
08:20 red-001 I would guess OS X uses more resources then thier linux setup
08:21 nerzhul yeah
08:21 red-001 oh I see looks like they can/are allowed only 180 active OS X VM's at a time
08:21 nerzhul it's not just a docker container
08:23 red-001 and they have a backlog in the thousands of builds
08:24 red-001 nerzhul, do we use the container for the linux or a full vm?
08:27 nerzhul it's always container on linux on travis if i remember
08:27 nerzhul but you cannot choose which
08:28 nerzhul on gitlab CI we use various different container
08:34 red-001 looks like you can choose for Ubuntu Trusty but not for Precise or OS X
08:35 nerzhul yes
08:37 red-001 looks like we use the full vm for all the builds
09:58 Gael-de-Sailly joined #minetest-dev
10:22 Krock joined #minetest-dev
10:23 Darcidride joined #minetest-dev
10:46 red-001 #6072 #6067
10:46 ShadowBot https://github.com/minetest/minetest/issues/6072 -- [CSM] Add functions to create particles and particlespawners. by red-001
10:46 ShadowBot https://github.com/minetest/minetest/issues/6067 -- [CSM] Add basic HUD manipulation. by red-001
11:09 nerzhul red-001, the chat6 submodule is an error no ?
11:11 red-001 yes
11:11 red-001 which one did I add that too?
11:12 nerzhul you also added your cmake configuration files :)
11:18 red-001 fixed
11:18 red-001 added the files to gitignore
11:42 paramat joined #minetest-dev
11:51 nerzhul red-001, approved both PR
11:51 nerzhul we need a second approval and we will have those useful features
12:05 Krock red-001, hud.cpp is missing in the android makefile
12:06 * red-001 grumbles something about auto-generating files
12:11 red-001 Krock, fixed
12:47 red-001 Krock, fixed your remaining issue with #6067
12:47 ShadowBot https://github.com/minetest/minetest/issues/6067 -- [CSM] Add basic HUD manipulation. by red-001
12:50 Krock 200
12:53 red-001 505
13:08 Fixer joined #minetest-dev
13:09 nerzhul merging #6067
13:09 ShadowBot https://github.com/minetest/minetest/issues/6067 -- [CSM] Add basic HUD manipulation. by red-001
13:09 nerzhul thanks red-001
13:10 nerzhul macosx build is stuck
13:11 red-001 there is a backlog of close to 3000 builds last I checked
13:12 red-001 so I imagine it will be like that for a while
13:14 CBugDCoder joined #minetest-dev
13:18 nerzhul Krock, #6884 is now fixed (we will see if it's okay on travis)
13:18 ShadowBot https://github.com/minetest/minetest/issues/6884 -- Game refactor: reduce game.cpp size by moving objects in the right places by nerzhul
13:30 CBugDCoder joined #minetest-dev
13:48 red-001 nerzhul, can you cancel https://travis-ci.org/minetest/minetest/builds/331191378?utm_source=github_status&utm_medium=notification ?
13:48 red-001 oh nvm looks like they fixed builds not being canceled on force push
13:48 red-001 about time they did imo
14:06 srifqi joined #minetest-dev
14:17 ThomasMonroe joined #minetest-dev
14:26 Gael-de-Sailly joined #minetest-dev
15:22 paramat sfan5 sofar ShadowNinja are you ok with Shara as a new MTG dev?
15:25 nore paramat: fyi, sofar is not on this channel
15:25 red-001 who are the MTG devs anyways?
15:25 paramat yeah
15:25 red-001 I'm never quite sure of who is a core dev and who is a MTG dev
15:26 paramat it's ..
15:26 sfan5 paramat: I'm fine with that
15:26 paramat ok
15:26 sfan5 hmm maybe we should have a separate group for mtg-only devs: https://github.com/orgs/minetest/teams
15:26 paramat ^ yes
15:26 paramat i almost suggested that yesterday
15:27 paramat even if only for documentation
15:27 red-001 would be useful if you split collborators into two groups and have one for the minetest repo and the other group for the minetest_game repo
15:28 red-001 seems there is often a good bit of confusion due to this
15:28 paramat Pilz (absent), sfan, nor-e, Kroc-k, ruben-, me, so-far, shadow-ninj
15:28 red-001 and it's easily avoidiable
15:28 paramat so-far shadow-ninj are inactive
15:29 red-001 Pilz isn't around at all as far as I see
15:29 red-001 not for quite a while now
15:29 paramat in fact i think pilz is somewhat fired for absence :)
15:30 red-001 README.MD sure needs some work
15:30 red-001 it still calls itself readme.txt....
15:30 red-001 oh the readme file in MTG wasn't renamed
15:31 red-001 paramat, so 11 isn't an MTG dev?
15:32 paramat correct
15:32 Krock red-001, all devs are in the minetest group
15:34 red-001 Krock, that would seem logical but wanted to confirm
15:35 red-001 I was almost sure there was talk of a new dev a while back but I saw no-one new on that page
15:37 nerzhul merging #6884
15:37 ShadowBot https://github.com/minetest/minetest/issues/6884 -- Game refactor: reduce game.cpp size by moving objects in the right places by nerzhul
15:37 nerzhul red-001, i think README.md is too huge, maybe it should be splited to have only a description and links to specific sections in dedicated files
15:38 nerzhul paramat, i never see pilzadam  since 2 years on this channel
15:39 paramat but then, some devs are absent long term and we don't kick them, so i guess pilz remains a core dev but can be assumed not a MTG dev
15:40 red-001 also with all these CSM security related issues being marked as high priority and blocker then so should #5989
15:40 ShadowBot https://github.com/minetest/minetest/issues/5989 -- [CSM] Inventory changes are sent to server instead of being handled by a local callback.
15:40 red-001 it's a much larger issue then just reading nodes or sending text
15:40 nerzhul paramat, i don't think a very long time inactive coredev can be a coredev as in 2 years the project technic, politic can change many
15:41 nerzhul red-001, it's completely relalted to the whole formspec stuff :(
15:44 red-001 if hashing built-in at compile time is a blocker then that almost certainly should be too
15:44 nerzhul yes
15:44 nerzhul paramat, a 0.5.0/0.4.17 release date has been defined or not ?
15:46 red-001 as for hashing builtin just move the death formspec back to core or something if it's that important, hashing builtin is the most pointless security measure I heard of in a long time
15:48 red-001 the issue should be marked "can't-fix" and "non-trivial" judging by the precident set by #6114, but that's just my opinion
15:48 ShadowBot https://github.com/minetest/minetest/issues/6114 -- Detection of dirty clients
15:57 nerzhul please look at #6936 :) could be nice to have
15:57 ShadowBot https://github.com/minetest/minetest/issues/6936 -- Add minetest issue template by nerzhul
15:59 paramat so moving death formspec back will avoid having to verify builtin? if moved would we then revert part of https://github.com/minetest/minetest/commit/308bb69eefdfb2ff3b893c124ccf0321d99a1bfd ?
16:00 red-001 it's as simple as adding a fallback formspec or just remove that little piece of lua code and adding it to c++
16:03 red-001 there might be some other stuff you might need to change but all of it si trival compare to creating a cross platform way to get the hash for all the files in builtin at compile time and then refuse to run if they change by runtime
16:03 red-001 s/si/is
16:04 red-001 I mean at that stage you might as well just implement #6114, chances are that would be simpler to do
16:04 ShadowBot https://github.com/minetest/minetest/issues/6114 -- Detection of dirty clients
16:04 red-001 (getting the hash of a file at runtime vs at compile time)
16:12 paramat !tell nerzhul sorry, i didn't reply about release dates. nothing defined yet, we're hoping for lord fingle and others to work on android and get it reasonable before release, so maybe 2+ months?
16:12 ShadowBot paramat: O.K.
16:12 paramat hmm
16:14 red-001 has anyone sent him an email or something, he seems to have disappeared after that competition he was in ended.
16:14 paramat yes i'm in touch
16:14 paramat by PM on forum
16:14 paramat a few days ago he said 'soon'
16:37 Fixer joined #minetest-dev
17:40 CBugDCoder joined #minetest-dev
17:46 lisac paramat: maybe MT could have a small proprietary anti-cheat part
17:46 lisac which server needs to somehow detect in order to approve it as the official client
17:47 lisac for example, maybe it could be a function that takes in a number, then calculates and returns another number
17:47 lisac server generates a number, sends it to the client, which returns the result
17:47 lisac if they match, it's the real client
17:48 lisac if not, just kick the client.
17:48 lisac if you build your own Minetest, you can download the compiled anticheat library part
17:48 lisac and set your client to use it
17:50 CBugDCoder sounds like a good idea but hacked clients could set themselves to use it too so how would it do anything?
17:50 lisac oh
17:50 lisac lol right
17:50 lisac fml
17:51 lisac you could just build it in the official mt clients
17:51 lisac which prevents custom builds now.
17:51 lisac nah
17:51 lisac hey, why not render everything on server and just receive input from client? Make client just play a video server sends.
17:52 lisac cheating would be impossible. :P
17:52 CBugDCoder but that would be sooooooooo laggy especialy on popular servers
17:53 pgimeno not to mention the bandwidth requirements
17:53 nore <lisac> paramat: maybe MT could have a small proprietary anti-cheat part <-- this is not happening
17:53 nore and won't be
17:54 lisac nore: yes, I'm just thinking out loud
17:54 nore minetest is an open-source project, and we won't force to include proprietary code in it
17:54 lisac or maybe we could have a better server side anticheat
17:54 nore this is the correct solution
17:55 lisac what are the most used cheats currently?
17:55 lisac Probably flying and noclip. Easiest to implement
17:55 lisac Flying is already really slowed down by the anticheat: very annoying to use.
17:55 lisac same for running.
17:56 lisac Some servers damage you while you are in walls without noclip priv.
17:58 Hijiri #6688
17:58 ShadowBot https://github.com/minetest/minetest/issues/6688 -- Custom particle generators for particle spawners by raymoo
18:01 lisac nore: maybe check if there is a node under player in PlayerSAO::checkMovementCheat(), then if the player is falling, or climbing, or has fly priv, and that way find out if he is fly-cheating?
18:03 Shara lisac: anticheat does nothign against flying from what I know. It just interferes with fast, which is not the same thing.
18:03 lisac Shara: try flying around without fast. It's annoying. :P
18:03 lisac but yes, still possible to cheat.
18:03 Shara That's your (and my) personal preference, but when you don't have fast anyway, anying already grants a huge advantage
18:04 Shara flying*
18:04 lisac I agree. We need a server side anti-fly
18:04 lisac or anti-fly-cheat
18:07 Shara Noclip is probably the biggest issue when it comes to cheating, at least from my experience (breaking into other players homes or safe areas when PVP is enabled)
18:07 nerzhul joined #minetest-dev
18:07 Shara And you only need to fly for a moment, at any speed at all, to abuse that
18:07 lisac hey nerzhul
18:07 nerzhul hi lisac
18:08 lisac <red-001> has anyone sent him an email or something, he seems to have disappeared after that competition he was in ended.
18:08 lisac red-001 seems to have been looking for you, nerzhul
18:08 nerzhul paramat, 2 months sounds reasonable, i need to test android build at a time
18:08 nerzhul i'm there for a time
18:08 lisac Shara: you could make a mod that checks if player is inside nodes
18:08 Raven262 NO, he wasn't looking for nerzhul.
18:08 Raven262 Read the logs again.
18:08 lisac if he is inside nodes a lot of times in a short time, kick him
18:08 Shara Yes, but think how frequently that has to check to actually be useful
18:08 lisac Raven262: thanks I'll check
18:09 nerzhul Raven262, if he needs me he will talk to me, then no real discussion there: p
18:09 Raven262 Just saying, sorry.
18:09 Shara And there's enough legitimate reasons (falling nodes dropping on players) for that
18:10 lisac Shara: which is why you check for a lot of nodes in a short period, like 5 nodes in 10 seconds
18:10 lisac either the player is drowning in bugged sand, or he is hacking.
18:10 lisac bonus if you check if the positions of the five nodes are different
18:10 Shara Still lots of checks. Too many on a busy server
18:11 pgimeno anti-fly might be as simple as: check how many nodes under the player are air (stop at first non-air node)
18:11 Shara And it won't even detect the issue I mainly have
18:11 nerzhul paramat, Krock sfan5 rubenwardy nore , can you all look at #6936, it's an issue template to help our users to fill issues, it's basic and can be improved before merge or after
18:11 ShadowBot https://github.com/minetest/minetest/issues/6936 -- Add minetest issue template by nerzhul
18:13 lisac pgimeno_: what about ladders and jumping?
18:13 lisac and falling?
18:14 Shara And trampolines that bounce players more than normal, and gravity changes and so on. Lots of things make it not so simple
18:15 lisac Shara: what about kick/ban votes?
18:15 lisac have a mod that allows players to call a vote to kick or ban another player
18:15 lisac would work for the time being
18:15 lisac until proper anti cheat
18:15 Shara That's not really relevant to the problem, and introduces lots of issues of its own
18:16 lisac it is the most basic form of anti cheat
18:16 lisac have other players report the issue
18:16 pgimeno_ lisac: ladders are nodes you're into. Jumping is not a problem. Falling is not a problem either, just detect constant or increasing height. Trampolines are the real problem here IMO. The mod should have a function to override the anti-cheat.
18:16 paramat ok
18:17 Shara It's not a fix for the actual issue. Andit's also something I'll never implement due to all the other problems (also I think not relelvant for -dev)
18:17 lisac pgimeno_: Don't trampolines just set the players gravity to another value?
18:18 paramat #6217 is server validation of client movement and is progressing
18:18 Shara I'd love a solution for fly/noclip cheating, but a mod won't cut it
18:18 ShadowBot https://github.com/minetest/minetest/issues/6217 -- Step height: Add as a player object property by paramat
18:18 paramat agh
18:18 paramat #6219
18:18 ShadowBot https://github.com/minetest/minetest/issues/6219 -- WIP: Server side movement by bendeutsch
18:19 Shara Glad he is still working on that
18:20 nerzhul paramat, i doesn't want to hurry up for 6219 it's a very sensible thing to code server side and network latency doesn't help
18:23 pgimeno_ lisac: I don't know. If that's the case, then it would not be a problem, as the number of nodes under the played can easily take into account gravity. I assumed a trampoline was a one-time impulse, which breaks my assumptions. It can still be detected as a parabolic movement, but network latency can screw that up as nerzul notes.
18:23 lisac IIRC, Raven262's Warzone mod jumppads use physics overrides https://forum.minetest.net/viewtopic.php?t=15199
18:25 Raven262 There are two ways to do the trampoline effect. One is a grav override other is a node group.
18:26 Raven262 I have used the override, other way was unknown to me and it would limit me to a much weaker effect.
18:28 pgimeno_ hm, 6219 looks like reimplementing client physics server side, which is going to be slow - you could as well do all physics on the server and just send the result to the client, which is what SL does
18:28 lisac SL==Second Life?
18:29 pgimeno_ yes
18:29 lisac I found the servers I tried a bit laggy
18:29 lisac not that I played more than 20 minutes of it
18:29 rubenwardy pgimeno_, client-side prediction, server-side reconcilliation
18:30 Fixer joined #minetest-dev
18:30 rubenwardy both sides should do player physics, but the server should take priority and correct and miscalculations client-side
18:39 Krock nerzhul, the issue template should also say something like "I have checked whether there are duplicate issues"
18:52 lisac and probably 'I checked if my build is clean'
18:52 nerzhul Krock, please add comment on the PR i will improve it :p
18:59 paramat sfan5 does #6072 have your approval now?
18:59 ShadowBot https://github.com/minetest/minetest/issues/6072 -- [CSM] Add functions to create particles and particlespawners. by red-001
18:59 sfan5 I haven't tested, but the code is fine
19:05 antims joined #minetest-dev
19:42 ThomasMonroe joined #minetest-dev
19:48 YuGiOhJCJ joined #minetest-dev
19:58 CBugDCoder joined #minetest-dev
20:45 Gael-de-Sailly joined #minetest-dev
21:24 clavi joined #minetest-dev
21:31 red-001 #6940
21:31 ShadowBot https://github.com/minetest/minetest/issues/6940 -- Fix issues with earlier CSM HUD commit by red-001
21:44 torgdor joined #minetest-dev
22:11 wayward1 joined #minetest-dev
22:13 ThomasMonroe joined #minetest-dev
22:20 Wayward_One joined #minetest-dev
22:23 CBugDCoder joined #minetest-dev
22:30 nerzhul merging #6936
22:30 ShadowBot https://github.com/minetest/minetest/issues/6936 -- Add minetest issue template by nerzhul
22:30 nerzhul and #6072
22:31 ShadowBot https://github.com/minetest/minetest/issues/6072 -- [CSM] Add functions to create particles and particlespawners. by red-001
22:32 sofar joined #minetest-dev
22:51 torgdor joined #minetest-dev
22:55 red-001 #6939
22:55 ShadowBot https://github.com/minetest/minetest/issues/6939 -- [CSM] on_connect callback is broken.
22:55 nerzhul red-001, #6941
22:55 nerzhul :p
22:55 ShadowBot https://github.com/minetest/minetest/issues/6941 -- [CSM] Remove `on_connect` callback by red-001
22:56 nerzhul sfan5, if you are okay i merge this trivial removal
22:56 nerzhul as this callback is useless since flavours as it cannot be triggered
22:59 sofar is core.localplayer then always valid?
23:00 nerzhul yes
23:00 nerzhul if i remember it's set when the CSM is inited
23:01 nerzhul before any mod loading
23:01 red-001 yep it was set just before the on_connect callback was called
23:01 red-001 https://github.com/minetest/minetest/pull/6941/files#diff-34f48ad91ac6c202ac60b0348ae90e30L1700
23:11 ThomasMonroe joined #minetest-dev
23:49 ThomasMonroe joined #minetest-dev

| Channels | #minetest-dev index | Today | | Google Search | Plaintext