IRC logs for #openttd on OFTC at 2024-03-17
            
00:01:27 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #10544: Fix #5713: Use pathfinder to find closest ship depot https://github.com/OpenTTD/OpenTTD/pull/10544
00:03:17 <xarick> no more bytes
00:36:17 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
01:11:01 *** charlieatlas_ff03k64 has joined #openttd
01:11:01 <charlieatlas_ff03k64> I just read the newest blog post. Would it be possible to somehow do cloud saves via personal cloud services? Like somehow link to Google drive? (Not asking if it is simple or anything, just possible)
01:14:19 <kamnet> I was just thinking this, why not put that back on the player? Or if OpenTTD is going to offer it on their own servers, offer it as a paid upgrade. That ust seems like a very unnecessary expense for a FOSS project to be burdened with.
01:16:37 <kamnet> Years ago I simply set up a symlink so that all of my saves are put in my Google Drive folder and automatically synced to all my computers. I have over 1GB in saves.
01:19:12 *** reldred has joined #openttd
01:19:12 <reldred> charlieatlas_ff03k64: I do this with NextCloud. Whole JGRPP folder (could do it with OTTD) thrown into a folder on nextcloud, my configs, saves, content downloaded, everything.
01:19:22 <reldred> Google Drive/Onedrive would work fine.
01:19:35 *** tabytac has joined #openttd
01:19:35 <tabytac> Yeah i do this with Syncthing ,a simmilar tool
01:48:25 *** tokai has joined #openttd
01:48:25 *** ChanServ sets mode: +v tokai
01:49:39 *** gelignite has quit IRC (Quit: Stay safe!)
01:55:06 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
03:20:52 *** Wormnest has quit IRC (Quit: Leaving)
03:30:50 *** debdog has joined #openttd
03:34:16 *** D-HUND has quit IRC (Ping timeout: 480 seconds)
03:49:46 *** Flygon_ has quit IRC (Remote host closed the connection)
04:02:42 *** gnu_jj has joined #openttd
04:05:51 *** gnu_jj_ has quit IRC (Ping timeout: 480 seconds)
04:40:37 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/6c5a8f55df8113b294651e63784825fd0b8bd417
04:40:38 <DorpsGek> - Update: Translations from eints (by translators)
05:02:11 *** keikoz has joined #openttd
06:28:50 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12300: Feature: encrypted game sockets https://github.com/OpenTTD/OpenTTD/pull/12300
06:34:46 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12303: Fix #12301: Update ship current order destination tile when relocating buoys https://github.com/OpenTTD/OpenTTD/pull/12303#pullrequestreview-1941383120
06:40:07 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #12315: Codechange: Iterate order lists instead of vehicles to find if any vehicle visits a station. https://github.com/OpenTTD/OpenTTD/pull/12315#pullrequestreview-1941383841
07:03:27 <merni> Lmao, the github website's file display doesn't support Tamil properly
07:04:16 <merni> https://cdn.discordapp.com/attachments/1008473233844097104/1218817164128026675/image.png?ex=66090af0&is=65f695f0&hm=ff6847151855724af419a515280b12414e58a5f3ca121ea66f41cb4985d94484&
07:05:26 <merni> by properly I mean "at all"
07:09:53 <merni> hum, it might be a font issue instead
07:10:07 <merni> works on edge
07:58:42 *** gelignite has joined #openttd
08:33:33 *** nielsm has joined #openttd
09:09:32 <DorpsGek> [OpenTTD/OpenTTD] nielsmh commented on discussion #12197: Vehicle window - show the most important information on top https://github.com/OpenTTD/OpenTTD/discussions/12197
09:13:32 <_zephyris> kamnet: It's a nice idea, but ugh, imagine the admin.
09:13:48 <DorpsGek> [OpenTTD/OpenTTD] nielsmh commented on discussion #12266: How to manipulate StringIDs? https://github.com/OpenTTD/OpenTTD/discussions/12266
09:14:46 <_zephyris> Helping third party solutions would be more reasonable, eg. detect and point to Dropbox, Onedrive, etc. if found.
09:15:05 *** Wolf01 has joined #openttd
09:15:54 <_zephyris> There is also that steam cloud save doc. which gets posted occasionally. Might be possible relatively easily
09:17:39 <truebrain> If only they would have a single interface .. that would be nice πŸ˜„
09:18:08 <_zephyris> Yeah
09:18:28 <_zephyris> Detecting 3 services on 3 OSs sounds really annoying!
09:19:01 <truebrain> And worse, if they upgrade and break support, we get the blame πŸ˜› and with our slow release cycle ....
09:19:15 <truebrain> Well, guess the plugin system can assist there
09:20:49 <_zephyris> Is the default save location easily configurable? That's the dumb solution
09:21:02 <truebrain> kamnet: About the pair part, I do not see a world where sufficient people would be willing to pay for such service, given how it is "free via ...".
09:21:53 <truebrain> _zephyris: What we still need to do, but nobody has done so, is to figure out what to save exactly
09:22:39 <_zephyris> Oh yeah, the settings/save debate.
09:22:44 <truebrain> For example, config file too? NewGRFs? Do saves need to keep a history, so you can also recover an older version? Etc etc
09:23:47 *** Ox7C5 has joined #openttd
09:23:51 <truebrain> It just requires someone writing something down, and then shape it from there. But I mentioned that a few times over the years already, but nobody so far has done so πŸ™‚
09:24:19 <truebrain> (As in, it isn't difficult; it does need doing)
09:25:13 <_zephyris> To github discussion!
09:25:47 <_zephyris> (brunch first)
09:26:19 <truebrain> Or a gist, or whatever πŸ™‚
09:26:29 <peter1138> brcoffee
09:26:54 <truebrain> But 14.0 first, so we can see how big saves are πŸ˜„
09:27:12 <truebrain> could also have stored the savegame compression algorithm
09:27:13 <truebrain> owh well
09:27:39 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320
09:30:21 <xarick> I use lzma:9 most of the time
09:34:31 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #12315: Codechange: Iterate order lists instead of vehicles to find if any vehicle visits a station. https://github.com/OpenTTD/OpenTTD/pull/12315
09:35:22 <DorpsGek> [OpenTTD/OpenTTD] eriksorngard opened issue #12324: [Bug]: Default custom currency not translatable https://github.com/OpenTTD/OpenTTD/issues/12324
09:38:14 <truebrain> ghehe; he is not wrong, but that is hardcoded as default in the settings file πŸ™‚
09:38:15 <DorpsGek> [OpenTTD/OpenTTD] merni-ns commented on discussion #12266: How to manipulate StringIDs? https://github.com/OpenTTD/OpenTTD/discussions/12266
09:38:20 <truebrain> don't think we can support translations there πŸ™‚
09:39:05 <peter1138> Unplayable.
09:40:03 <DorpsGek> [OpenTTD/OpenTTD] merni-ns commented on discussion #12266: How to manipulate StringIDs? https://github.com/OpenTTD/OpenTTD/discussions/12266
09:40:45 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323
09:43:13 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on issue #12324: [Bug]: Default custom currency not translatable https://github.com/OpenTTD/OpenTTD/issues/12324
09:47:59 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12300: Feature: encrypted game sockets https://github.com/OpenTTD/OpenTTD/pull/12300
09:54:52 <truebrain> haha, Steam forums has the first OMG SURVEY STUFF IS THE WORST THING EVER SPYWARE BLOAT !!!!!
09:54:53 <truebrain> haha
09:56:15 <reldred> inb4 survey cpu lag is the new pbs cpu lag
09:56:39 <michi_cc> Well, at least there are comments. Reddit is completely dead this weekend.
09:58:48 <truebrain> I am trying out GitButler, which sounds like an interesting concept, with virtual branches. Just ... the app is using 1300% CPU ... feels a bit excessive πŸ˜„
09:59:47 <locosage> are there still known bugs with house cargo acceptance in 14.0-RC2?
10:00:29 <locosage> coz it looks broken
10:02:31 <truebrain> hmm ... I have zero (!) survey reports from RC2 so far
10:02:35 <truebrain> that seems .. suspecious
10:03:01 <locosage> https://cdn.discordapp.com/attachments/1008473233844097104/1218862145018794105/Screenshot_from_2024-03-17_17-02-43.png?ex=660934d4&is=65f6bfd4&hm=6c831004548ff727ff9ecb12fefbbd04b53463edbb49b78edc9d4cc024a64488&
10:03:01 <locosage> same game, same house
10:05:21 <truebrain> oops .. that is my mistake, no RC2 was coming through
10:05:21 <truebrain> lol
10:05:31 <truebrain> happy I check πŸ˜„
10:07:36 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #12325: Fix #12302: Allow empty train engines to use an invalid cargo type. https://github.com/OpenTTD/OpenTTD/pull/12325
10:09:48 <truebrain> that commit message looked funny
10:09:59 <peter1138> I think it has a typo or two in it.
10:10:00 <truebrain> the description tells the commit message makes total sense πŸ˜„
10:10:02 <truebrain> nice write-up πŸ™‚
10:11:48 <truebrain> how do you make a multi-line suggestion in GitHub ..
10:12:01 <truebrain> found it!
10:12:44 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12325: Fix #12302: Allow empty train engines to use an invalid cargo type. https://github.com/OpenTTD/OpenTTD/pull/12325#pullrequestreview-1941425705
10:13:40 <salut3585> https://cdn.discordapp.com/attachments/1008473233844097104/1218864827582124112/image.png?ex=66093754&is=65f6c254&hm=21ea4063a0c91020a46d787defd3f58e17daa45dccbc3a1cc58c12875479c775&
10:13:40 <salut3585> it it possible to add e new key like this in OpenGFX
10:15:07 <peter1138> No, it needs to be added to the game code, and to nml first.
10:15:34 <peter1138> And OpenGFX is just a one of a few basesets, not everyone uses it.
10:18:53 <talltyler> frosch123: That’s why wallclock mode subsidies and timetables show time remaining relative to the current moment. Subsidies shows minutes remaining, which is probably what you want. πŸ™‚
10:19:23 <salut3585> peter1138: I've added new sprites to media/baseset/openttd.grf.
10:19:23 <salut3585> but when I start openttd it doesn't work, it wants to have opengfx sprites, if I don't have opengfx sprites in openttd it won't start
10:20:39 <michi_cc> As openttd.grf is always loaded with all basesets, you very likely have an error somewhere else.
10:21:11 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick commented on pull request #12303: Fix #12301: Update ship current order destination tile when relocating buoys https://github.com/OpenTTD/OpenTTD/pull/12303#pullrequestreview-1941427635
10:21:16 <andythenorth> hmm...upgrade my mac day?
10:21:25 <andythenorth> such things
10:21:32 <michi_cc> What could possibly go wrong? πŸ˜›
10:21:37 <reldred> m3 air looks nice
10:21:43 <andythenorth> I mean the OS πŸ˜›
10:21:48 <reldred> Oh, lmao
10:21:58 <andythenorth> so I can compile OpenTTD, and not be reduced to downloading binaries like a peon
10:22:11 <andythenorth> I guess I make a backup first
10:22:28 <reldred> I only compile for linux arm64 πŸ˜›
10:22:34 <reldred> otherwise precompiled bins are fine
10:24:58 <peter1138> locosage: I'm sure you're aware this a terrible example of a bug report.
10:26:06 <DorpsGek> [OpenTTD/survey-web] TrueBrain opened pull request #45: Add: accept surveys with schema version 2 https://github.com/OpenTTD/survey-web/pull/45
10:26:45 <locosage> peter1138: Yeah, I'll make a proper one
10:26:55 <salut3585> michi_cc: Do you mean if I clone openttd from git and compile it? it will work without downloading resources from the Internet, right?
10:27:16 <locosage> just wanted to make sure it's not something known as there were lots of changes in cargo acceptance I didn't pay much attention to
10:27:37 <DorpsGek> [OpenTTD/survey-web] TrueBrain opened pull request #46: Add: analyze surveys with schema version 2 https://github.com/OpenTTD/survey-web/pull/46
10:27:48 <peter1138> If you're not building your own version of openttd, why are you adding a new sprite?
10:28:18 <salut3585> peter1138: I want to add a new button
10:28:21 <michi_cc> salut3585: I might have misunderstood you. There's no point in adding any *new* sprites to openttd.grf if you are not developing a source patch. You might modify some, but usually a NewGRF is way easier.
10:30:46 <DorpsGek> [OpenTTD/survey-web] TrueBrain updated pull request #46: Add: analyze surveys with schema version 2 https://github.com/OpenTTD/survey-web/pull/46
10:30:59 <DorpsGek> [OpenTTD/survey-web] TrueBrain merged pull request #45: Add: accept surveys with schema version 2 https://github.com/OpenTTD/survey-web/pull/45
10:31:59 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326
10:33:01 <peter1138> TrueBrainviaGitHub: oops πŸ™‚
10:33:19 <truebrain> well, #45 was the boo-boo πŸ˜›
10:33:34 <truebrain> I was working on #46 so tonight the survey analyzer will run fine πŸ˜›
10:33:43 <truebrain> but all good, it is deployed now πŸ™‚
10:34:25 <Rubidium> michi_cc: I think the context is adding a new button to the landscape menu
10:34:46 <DorpsGek> [OpenTTD/OpenTTD] ldpl opened issue #12327: [Bug]: House cargo acceptnance changes when loading the game in newer OpenTTD version https://github.com/OpenTTD/OpenTTD/issues/12327
10:36:40 <salut3585> michi_cc: I wanted to add new functionality with new buttons and I need to add new sprites and icons.
10:36:40 <salut3585> Why should I create a patch? I have git openttd and I change the code in it, then I open the PR and if you like it, you can merge it.
10:36:40 <salut3585> And I thought that openttf uses openttd.grf, but then I realized that openttd downloads the opengfx package with sprites.
10:36:40 <salut3585> I also added sprites here.
10:36:40 <salut3585> But I don’t see them in memory after launch.
10:36:41 <salut3585> And I understand that I'm doing something wrong
10:37:17 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #12303: Fix #12301: Update ship current order destination tile when relocating buoys https://github.com/OpenTTD/OpenTTD/pull/12303
10:38:41 <peter1138> Damn it, why is every broken save 4k x 4k.
10:39:06 <locosage> coz ppl play it :p
10:39:27 <DorpsGek> [OpenTTD/survey-web] TrueBrain merged pull request #46: Add: analyze surveys with schema version 2 https://github.com/OpenTTD/survey-web/pull/46
10:39:38 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1218871362643562536/image.png?ex=66093d6a&is=65f6c86a&hm=278485bda8de987091fa1e344209ef4269b934056e0c69a75b9f2ea12650cde2&
10:39:38 <xarick> what?
10:40:41 <locosage> and if you're looking at #12327 that map is for the challenge that lasts like 100 years or smth
10:40:51 <xarick> I don't remember my 13.4 having that πŸ˜„
10:40:53 <locosage> though it seems to die to game performance much sooner
10:40:58 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941432012
10:41:54 <locosage> xarick: that's cmclient on the left side, I just didn't bother compiling vanilla for the screenshot
10:42:29 <peter1138> I think it's a bug that's fixed.
10:42:50 <locosage> well, on the right is a relatively fresh master
10:42:50 <peter1138> TTRS has a translation table of PASS, PETR, MAIL, TOUR, FOOD and GOOD.
10:43:04 <peter1138> No, I mean it accepting those cargos on the left is a bug.
10:43:33 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#issuecomment-2002402947
10:43:41 <locosage> oh...
10:44:12 <peter1138> Hmm, do I need a debug build to show landarea output in the console?
10:45:18 <Rubidium> peter1138: yes, or you have to change the code a bit
10:45:19 <_jgr_> It should print at debug level misc=1 in normal builds
10:45:38 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #12327: [Bug]: House cargo acceptnance changes when loading the game in newer OpenTTD version https://github.com/OpenTTD/OpenTTD/issues/12327
10:45:56 <peter1138> _jgr_: Ah, thanks.
10:46:21 <michi_cc> salut3585: openttd.grf is loaded before the baseset, so added GUI sprites do not need to be added to any other baseset. If that doesn't work for you, we're back at the start: you probably have an error in your code somewhere.
10:46:38 <michi_cc> OpenGFX may of course also add a specific sprite for the button, but it should not be required.
10:51:00 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #12303: Fix #12301: Update ship current order destination tile when relocating buoys https://github.com/OpenTTD/OpenTTD/pull/12303
10:52:08 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick commented on pull request #12303: Fix #12301: Update ship current order destination tile when relocating buoys https://github.com/OpenTTD/OpenTTD/pull/12303#pullrequestreview-1941434000
10:53:41 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323#pullrequestreview-1941432804
10:54:20 <truebrain> Rubidium: lol @ branch name of 12323 πŸ˜„
10:55:38 <peter1138> ffs, I just can't debug with sdl2... how is it meant to work?
10:55:47 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941434690
10:55:52 <andythenorth> oof backing up is slow
10:56:17 <andythenorth> and it's only 1 TB, because no real human can afford more than 1 TB of storage in an Apple device
10:57:02 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941434893
10:58:36 <andythenorth> time for biscuits?
10:59:03 <andythenorth> ach thunderbolt nvme enclosures are still silly money
10:59:05 <andythenorth> Β£90 +
10:59:27 <andythenorth> USB 3.2 are aboutΒ£15
11:01:14 <andythenorth> hmm 3.2 enclosures, but only 3.1 protocol?
11:01:21 <andythenorth> seems a bit lol
11:02:00 <peter1138> Okay, nearly finished rebuilding with SDL1.
11:03:52 <_jgr_> Are you using gdb or something else?
11:04:09 <_jgr_> Debugging SDL2 builds works fine here
11:04:52 <peter1138> lldb.
11:05:06 <peter1138> Debugging works, but SDL2 has grabbed the mouse cursor.
11:05:07 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941436635
11:05:43 <peter1138> So I can move the mouse outside OpenTTD (and it's frozen because it reached a break-point) nor click on anything.
11:05:49 <peter1138> ...
11:05:52 <peter1138> *CAN'T*
11:06:13 <andythenorth> naughty
11:06:14 <peter1138> With SDL1 when it hits a breakpoint the mouse cursor is magically released.
11:07:20 <andythenorth> hmm I had 150 GB HD free. Dropbox was using 200 GB locally. I moved 120 GB of that to 'online only'. Now I have 95 GB of local HD. Maths is weird.
11:09:44 <truebrain> Rubidium: okay, joining a password protected server without entering a password feels like magic πŸ˜„
11:09:50 <truebrain> (in a positive way, ofc)
11:12:55 <truebrain> okay, sometimes you do get back "wrong password" which isn't actually true .. you are not whitelisted πŸ˜›
11:15:43 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323#pullrequestreview-1941438387
11:16:49 <truebrain> Rubidium: you really did a nice job, with the abstraction and mostly: with the handlers and the way they are being processed. It is (relatively, for security shit) easy to read and code-wise good to follow. Seriously, nice job there πŸ™‚
11:17:50 <Rubidium> about the wrong password, I guess that's when the server has no password set but has authorized keys set?
11:18:19 <truebrain> yeah, I wrote out the scenarios in my comment; that is the one that is confusing
11:18:50 <truebrain> (well, confusing from the user perspective; it does exactly what I expected it to do πŸ˜› )
11:20:12 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #12327: [Bug]: House cargo acceptnance changes when loading the game in newer OpenTTD version https://github.com/OpenTTD/OpenTTD/issues/12327
11:20:34 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12300: Feature: encrypted game sockets https://github.com/OpenTTD/OpenTTD/pull/12300#pullrequestreview-1941439423
11:23:22 <truebrain> Rubidium: am I right to assume/understand that the server generates a new private/public key on every start?
11:26:08 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320
11:27:01 <Rubidium> truebrain: it creates a new set on every authentication
11:27:12 <truebrain> ah; good
11:27:20 <truebrain> was hard to spot where that happened πŸ˜‰
11:27:46 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323#pullrequestreview-1941441476
11:31:56 <truebrain> can you actually do in C++: `void a(b = c())` ?
11:33:25 <peter1138> If c() is constexpr, yes.
11:33:40 <truebrain> well, it doesn't have to be a constexpr explicitly, it seems
11:33:43 <peter1138> And maybe if it isn't, but not sure πŸ™‚
11:33:55 <truebrain> https://godbolt.org/z/hPajzKv13 is working ..
11:34:21 <truebrain> I am not sure what to think of that being allowed .. it feels really weird πŸ˜›
11:39:48 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941445351
11:45:27 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941446625
11:45:39 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323#pullrequestreview-1941446655
11:46:13 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941446764
11:46:34 <DorpsGek> [OpenTTD/OpenTTD] ldpl commented on issue #12327: [Bug]: House cargo acceptnance changes when loading the game in newer OpenTTD version https://github.com/OpenTTD/OpenTTD/issues/12327
11:46:50 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323#pullrequestreview-1941446840
11:54:14 <andythenorth> and now I have 272 GB free πŸ˜›
11:54:23 <andythenorth> I guess macOS has some special rules for when to free space πŸ˜›
11:54:36 <peter1138> Java is broken on macOS 14.4 apparently.
11:54:55 *** marchnex has joined #openttd
11:57:42 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323#pullrequestreview-1941449564
11:59:00 <truebrain> Who needs Java anyway
12:02:50 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #12325: Fix #12302: Allow empty train engines to use an invalid cargo type. https://github.com/OpenTTD/OpenTTD/pull/12325#pullrequestreview-1941450756
12:06:15 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323
12:06:18 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320
12:08:52 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323
12:12:52 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323#pullrequestreview-1941452693
12:13:55 <truebrain> `==48690== More than 1000 different errors detected. I'm not reporting any more.` .. all 1000 are in WSL stuff πŸ˜›
12:14:30 <truebrain> `==48690== Final error counts will be inaccurate. Go fix your program!` haha, yeah, nvidia, go fix your driver! πŸ˜›
12:16:11 <johnfranklin> truebrain: Minecraft players?
12:16:40 <truebrain> okay, `--error-limit=no` fixes that issue .. silly WSL nvidia driver
12:21:29 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941454897
12:21:32 <DorpsGek> [OpenTTD/OpenTTD] TheMowgliMan commented on discussion #12266: How to manipulate StringIDs? https://github.com/OpenTTD/OpenTTD/discussions/12266
12:23:19 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323
12:24:32 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323
12:24:46 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941455509
12:26:12 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941457094
12:43:19 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain opened pull request #12328: Codechange: the "no revision detected" string is with four zeros (norev0000) https://github.com/OpenTTD/OpenTTD/pull/12328
12:45:51 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941466257
12:47:15 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #12328: Codechange: the "no revision detected" string is with four zeros (norev0000) https://github.com/OpenTTD/OpenTTD/pull/12328#pullrequestreview-1941466547
12:47:28 <truebrain> Rubidium: why do you generate a new key for clients too? (as in, what is your thinking here, for my understanding)
12:53:26 *** drludde has joined #openttd
12:53:26 <drludde> Orudge: nice to see you've been active all those years and helping the project stay alive πŸ™‚ I'm only like 5% active of the total project lifetime now
12:53:32 <Rubidium> I have not put a lot of thought into why. It's simply less code
12:53:43 <truebrain> it is not that it is a bad thing btw
12:56:02 <Rubidium> though back in the initial days network_client.cpp and network_server.cpp still included network_crypto_internal.h and did not have authorized keys, so there wasn't really a key yet
12:56:20 <truebrain> normally ofc you use the keys to setup the encryption too, in one go
12:56:39 <truebrain> but now you first use set A to validate if the client can go in, generate a new set to start encryption, if I understand you correct
12:57:01 <drludde> Why does it say Rubidium is a bot? Is chatgpt a new openttd dev? πŸ˜…
12:57:13 <truebrain> no, he is still on that weird network called IRC
12:57:20 <truebrain> we bridge between Discord and IRC πŸ™‚
12:58:00 <drludde> I stopped using irc after i realized that it's nice to be able to paste pictures
12:58:27 <drludde> πŸ˜…
12:59:34 <drludde> Those good old IRC days.... Interesting discussions every day during highschool
13:00:02 <drludde> Now it's just like a barren Fallout 4 landscape
13:00:17 <drludde> Sorry for being offtopic
13:01:05 <frosch123> truebrain: default arguments are evaluated at caller site on every call
13:01:17 <frosch123> that's also how the std::source_location magic works
13:01:27 <frosch123> so c++ is different from python here
13:01:53 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941468140
13:02:13 <drludde> b=c is just an expression like b+c
13:02:33 <Rubidium> truebrain: encryption of data is generally not done using a public/secret key system, and in the case of monocypher the encryption and decryption happen with the same secret. So you need to agree on the secret for the encryption somehow, and that's done with the key exchange where the secret keys for the encryption are derived from
13:02:40 <truebrain> frosch123: I wasn't comparing languages πŸ™‚ I simple never knew it was allowed, nor actively seen it somewhere being used πŸ˜›
13:02:50 <drludde> Python added := which seems to work similarly, at least you can use it in ifs
13:03:17 <truebrain> Rubidium: yeah, I know. The pub/sec is used to derive a secret used for the encryption
13:03:41 <frosch123> i am sure you have seen ``void foo(std::string a = std::string())`` or ``void foo(std::string a = "")`` somewhere
13:03:53 <frosch123> std::string constructor is also just a global function
13:04:00 <truebrain> the first, nope. The second, sure.
13:04:16 <truebrain> and it really is okay for me to not have seen something before, not knowing it was allowed πŸ™‚
13:04:19 <truebrain> these things happen πŸ˜‰
13:04:28 <drludde> C++
13:05:22 <Rubidium> truebrain: and the encryption uses the derived secret keys from the successful authentication. So no extra key exchange is done for the encryption keys
13:05:34 <truebrain> and that is the part that doesn't add up to me πŸ™‚
13:05:44 <truebrain> you mention that the client uses a fresh pub/sec for every connection
13:05:52 <truebrain> but somehow the authentication is done with the stored pub
13:05:55 <truebrain> so I need to read the code a bit better πŸ˜„
13:06:05 <drludde> The second one just looks like a type declaration. A(B=C()) looks like an expression although perhaps it's ambiguous?
13:06:20 <truebrain> owh, I need to read better .. you wrote: "the others"
13:06:23 <truebrain> I misread that in your comment
13:06:27 <truebrain> okay, that makes more sense πŸ˜›
13:07:05 <frosch123> truebrain: he, it's the opposite for me :p usually i use things for years without knowing what they really do, like ``def foo(a = [])``
13:07:08 <truebrain> Rubidium: okay, I recovered a bit of sanity. I misunderstood what you were writing. So it is working as I would expect, except that sometimes we magically generate pub/sec if it isn't set yet πŸ™‚
13:07:39 <truebrain> frosch123: that really should be in your Python 101 course; as that is a very common noob-trap (and the part of Python that is just bad designed, tbfh)
13:08:16 <_glx_> Oh yeah I fell into this trap when working in nml
13:09:32 <truebrain> Rubidium: when I have a client pub/sec, but I use a password to login, would it still generate a new pub/sec for the client when encryption starts?
13:12:45 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941472397
13:15:59 <kamnet> truebrain: Yes, I was thinking the plugin system would be perfect for that.
13:17:02 <Rubidium> truebrain: it would use the newly generated pub/sec for the password-authenticated key exchange which would yield the encryption keys if successful
13:17:17 <truebrain> that would be unwanted, I think, for future additions
13:17:23 <andythenorth> ok let's update xcode next πŸ˜›
13:17:34 <Rubidium> truebrain: why?
13:17:39 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #12328: Codechange: the "no revision detected" string is with four zeros (norev0000) https://github.com/OpenTTD/OpenTTD/pull/12328
13:17:52 <truebrain> as example, when I would make companies have a whitelist, based on pub-keys. So you join my game, I whitelist you to my company. Now you reconnect (via password), but can't rejoin the company, as your pub-key changed.
13:18:04 <truebrain> I would think that the pub-key of clients should be stable, no matter what method they use to connect
13:19:05 <truebrain> ugh, Discord: connect -> authenticate πŸ™‚
13:20:26 <Rubidium> take a look at #12326
13:21:40 <Rubidium> there it always uses the public key provided by the authorized key handler
13:22:52 <truebrain> in general, all these extra pub/sec that are created / can be created, doesn't make this easier to read
13:23:12 <truebrain> I wonder if we wouldn't make our lifes a bit easier / more readable, if we just assign a single pub/sec to a connection
13:23:38 <truebrain> so instead of feeding it to `X25519AuthorizedKeyClientHandler`, that we feed it to `CombinedAuthenticationClientHandler`
13:24:01 <andythenorth> hmm not bad, Horse compile time is same on this new macOS, even though 2 performance cores are tied up installing Xcode and running the AV
13:25:04 <_glx_> your hardware didn't change πŸ™‚
13:25:41 <Rubidium> I wouldn't mess with the Combined one, but rather the ::Create for client/server that just always need to pass the secret key to the X25519 variants
13:26:15 <Rubidium> that shouldn't be that difficult to achieve
13:28:44 <truebrain> haha, I leave details to you if you don't mind πŸ˜„
13:29:07 <truebrain> but yeah, that would simplify things a bit, as that way a client always has the same pub-key (from a server perspective)
13:31:53 <andythenorth> _glx_: it's rare, but not unknown for macOS to have performance regressions between versions πŸ˜›
13:33:40 <truebrain> Rubidium: yeah, okay, reading it over again, your suggestion would work yes. And for the server, in the Create() always generate a key, which gives it a perfect place to say: every connection has its own key
13:33:50 <truebrain> if we change our mind on that in some point in the future, changing that would be trivial
13:34:19 <truebrain> that at least to me would make it easier to deduce what is being used where πŸ™‚
13:34:45 <truebrain> and after one of the authentication methods is used, you get that handler back (proxies via Combined), where you can create encryption handlers
13:34:48 <truebrain> okay, sounds sane
13:56:02 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320
13:56:38 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941485333
13:57:55 <truebrain> nice! πŸ™‚
13:58:38 <truebrain> okay, I will give it one more review later tonight, but I think we are almost there πŸ˜„
13:58:49 <Rubidium> adding that extra error message really hurts development time ;(
13:58:52 <truebrain> maybe someone else also wants to look it over πŸ™‚
13:59:14 <truebrain> I have been experimenting with sccache
13:59:26 <truebrain> just to not get annoying by the countless rebuilds πŸ˜›
13:59:48 <truebrain> but, think of it like this: that extra error message really does look pretty πŸ™‚
14:01:05 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323
14:01:08 <kuhnovic> Not sure if I've asked this before, but do we have a function to select a random bit out of a bunch of set bits (e.g. TrackdirBits)
14:01:29 <andythenorth> wonder if vcpkg needs rebuilt for a new OS?
14:01:37 <andythenorth> cmake ran fine
14:01:55 *** Flygon has joined #openttd
14:03:31 <andythenorth> build works
14:05:55 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12300: Feature: encrypted game sockets https://github.com/OpenTTD/OpenTTD/pull/12300
14:06:29 <andythenorth> 1m 25s build from clean (release build, think it's ARM arch only)
14:10:27 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #12325: Fix #12302: Allow empty train engines to use an invalid cargo type. https://github.com/OpenTTD/OpenTTD/pull/12325
14:12:09 <andythenorth> Flat Invalid Cargos?
14:27:11 <_glx_> kuhnovic: PickRandomBit ?
14:28:13 <kuhnovic> That's what I needed, thanks!
14:29:00 <_glx_> andythenorth: it rebuilds deps on compiler change
14:29:29 <_glx_> but it's handled by cmake
14:31:30 <andythenorth> great πŸ™‚
14:32:13 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320
14:32:16 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326
14:32:19 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323
14:32:22 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12300: Feature: encrypted game sockets https://github.com/OpenTTD/OpenTTD/pull/12300
14:42:02 <DorpsGek> [OpenTTD/OpenTTD] tibenasri commented on issue #12312: [Bug]: towns build roads even when "town cannot build roads" is enabled https://github.com/OpenTTD/OpenTTD/issues/12312
14:43:53 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #11495: Change: Simplify sprite cache memory management https://github.com/OpenTTD/OpenTTD/pull/11495
14:47:27 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #12312: [Bug]: towns build roads even when "town cannot build roads" is enabled https://github.com/OpenTTD/OpenTTD/issues/12312
14:54:29 <DorpsGek> [OpenTTD/OpenTTD] tibenasri commented on issue #12312: [Bug]: towns build roads even when "town cannot build roads" is enabled https://github.com/OpenTTD/OpenTTD/issues/12312
15:03:10 <DorpsGek> [OpenTTD/OpenTTD] PeterN dismissed a review for pull request #12298: Codechange: Shuffle members to reduce vehicle size. https://github.com/OpenTTD/OpenTTD/pull/12298#pullrequestreview-1939412205
15:03:13 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #12298: Codechange: Shuffle members to reduce vehicle size. https://github.com/OpenTTD/OpenTTD/pull/12298
15:05:58 *** Wormnest has joined #openttd
15:12:22 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #12298: Codechange: Shuffle members to reduce vehicle size. https://github.com/OpenTTD/OpenTTD/pull/12298#issuecomment-2002504274
15:36:32 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320
15:40:44 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #11878: Add: public key authentication to the admin connection https://github.com/OpenTTD/OpenTTD/pull/11878
15:41:28 *** klote has joined #openttd
15:41:28 <klote> Does any one know who created the [GS] CityController v9 game script?
15:42:20 <frosch123> it's on bananas: https://bananas.openttd.org/package/game-script/43544354
15:43:51 <frosch123> you can contact them via tt-forums, or via github: https://github.com/primer97
15:44:25 <frosch123> they even host it there: https://github.com/primer97/ctct
15:45:12 <klote> ah thanks
15:48:36 <klote> i did both. hope he is still active
15:49:04 <klote> script isnt functionally fully late game
15:55:42 <DorpsGek> [OpenTTD/OpenTTD] tibenasri commented on issue #12312: [Bug]: towns build roads even when "town cannot build roads" is enabled https://github.com/OpenTTD/OpenTTD/issues/12312
16:05:31 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #12325: Fix #12302: Allow empty train engines to use an invalid cargo type. https://github.com/OpenTTD/OpenTTD/pull/12325#pullrequestreview-1941523524
16:13:40 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #12325: Fix #12302: Allow empty train engines to use an invalid cargo type. https://github.com/OpenTTD/OpenTTD/pull/12325#pullrequestreview-1941524881
16:17:50 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #12298: Codechange: Shuffle members to reduce vehicle size. https://github.com/OpenTTD/OpenTTD/pull/12298#pullrequestreview-1941525516
16:22:41 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #12298: Codechange: Shuffle members to reduce vehicle size. https://github.com/OpenTTD/OpenTTD/pull/12298
16:23:17 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #12325: Fix #12302: Allow empty train engines to use an invalid cargo type. https://github.com/OpenTTD/OpenTTD/pull/12325
16:23:20 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #12302: [Bug]: #11719 changes the availablity of some vehicles for some GRFs https://github.com/OpenTTD/OpenTTD/issues/12302
16:24:36 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #12313: Fix #12305: Crash with large positive sprite x offset in engine preview window https://github.com/OpenTTD/OpenTTD/pull/12313#issuecomment-2002529461
17:07:14 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed pull request #10768: Codechange: Clean up scrollbars by adding iteration https://github.com/OpenTTD/OpenTTD/pull/10768
17:07:17 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10768: Codechange: Clean up scrollbars by adding iteration https://github.com/OpenTTD/OpenTTD/pull/10768#issuecomment-2002541349
17:10:49 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1218969807152742543/image.png?ex=66099919&is=65f72419&hm=df851af2c91d5c236421b2272e6c83cd3d53df3c67fdbda2ab4a95966bc7f36b&
17:10:49 <xarick> shouldn't this auto resize to fit?
17:31:58 <truebrain> when you looked a PR over so many times, you can't actually look at it anymore .. YOLO? YOLO! πŸ™‚
17:32:18 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320#pullrequestreview-1941537604
17:32:22 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323#pullrequestreview-1941537613
17:33:57 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #12320: Codechange: add set of classes providing authentication and encryption https://github.com/OpenTTD/OpenTTD/pull/12320
17:34:50 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 dismissed a review for pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323#pullrequestreview-1941537613
17:34:53 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323
17:35:26 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323#pullrequestreview-1941538173
17:39:04 <truebrain> tcpdump tells me very clearly traffic is actually encrypted with #12300 πŸ˜›
17:39:17 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #12300: Feature: encrypted game sockets https://github.com/OpenTTD/OpenTTD/pull/12300#pullrequestreview-1941538718
17:39:37 <truebrain> the only thing that scares me, as this most likely also means I should make the coordinator connection encrypted πŸ˜›
17:39:58 <andythenorth> pff something is broken in my libpng πŸ˜›
17:41:38 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326#pullrequestreview-1941539233
17:41:54 <truebrain> funny how small such a PR all of a sudden is πŸ™‚
17:42:34 <_glx_> andythenorth: wasn't that already an issue you previously had ?
17:42:52 <andythenorth> maybe, might just try deleting old brew first
17:42:56 <andythenorth> then install clean
17:43:08 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326#pullrequestreview-1941539479
17:43:09 <andythenorth> wonder if I could teach python to use vcpkg
17:43:13 <_glx_> just use vcpkg it's really simpler
17:43:34 <_glx_> you clone, you tell cmake where it is, ant it just works
17:43:40 <_glx_> nothing to install
17:44:17 <_glx_> the libs are built inside openttd build dir
17:44:27 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326#pullrequestreview-1941539670
17:44:27 <andythenorth> not sure how to tell python to use vcpkg
17:44:37 <andythenorth> there are lots of libs needed for e.g. pillow
17:45:18 <_glx_> oh I don't know for python
17:45:54 <Rubidium> truebrain: there's still a long time to think about the coordinator connection ;) And with the admin port I'm using libottdadmin2 for testing, so you might get most of the required bits from there
17:46:24 <truebrain> true true πŸ™‚
17:47:07 <andythenorth> hmm, google is not giving me obvious answers πŸ™‚
17:47:18 <andythenorth> back to brew I guess
17:47:43 <truebrain> Rubidium: with merging of these PRs, btw don't forget to close the other PR dealing with encryption πŸ™‚
17:48:10 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326#pullrequestreview-1941540749
17:49:22 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326#pullrequestreview-1941541019
17:50:00 <andythenorth> hmm still broken with new brew
17:50:04 <Rubidium> I can't find a PR dealing with encryption, there is one with public key authentication for companies but that's not done yet
17:50:10 <andythenorth> do we have nml binaries for macOS?
17:50:41 <_glx_> no standalone macOS version
17:50:52 <andythenorth> libimagequant is failing, and there's nothing I could find in google about it https://gist.githubusercontent.com/andythenorth/58cd829fb9daa3c31e1bb314cc0e467c/raw/add2121e10ed08e68a65fd80cf013a34434ee588/gistfile1.txt
17:50:55 <truebrain> Rubidium: https://github.com/OpenTTD/OpenTTD/pull/8391
17:54:03 <Rubidium> can't close that yet as the main part of that, the public key authentication of companies has not been implemented (yet)
17:54:25 <truebrain> well, that PR is not going to happen now ofc; given it uses a different library
17:54:34 <truebrain> but sure, when ever you feel you can close it
17:55:44 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 dismissed a review for pull request #12300: Feature: encrypted game sockets https://github.com/OpenTTD/OpenTTD/pull/12300#pullrequestreview-1941538718
17:55:47 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12300: Feature: encrypted game sockets https://github.com/OpenTTD/OpenTTD/pull/12300
17:58:44 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326
18:00:55 <peter1138> lol, playing a Doom wad and it's using early prog-era Genesis as one of the music tracks.
18:06:47 <andythenorth> pff reinstalling python hasn't fixed this
18:06:54 <andythenorth> I thought maybe it had lost bindings to dll
18:06:59 <peter1138> 4 of its tracks, well.
18:07:44 <andythenorth> ach this does seem to be limited to pypy3
18:09:25 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #12323: Feature: authenticate to the server without sending the password https://github.com/OpenTTD/OpenTTD/pull/12323
18:09:49 <truebrain> w00p
18:12:33 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12300: Feature: encrypted game sockets https://github.com/OpenTTD/OpenTTD/pull/12300
18:12:58 <Rubidium> all the conflicts :(
18:14:37 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #12300: Feature: encrypted game sockets https://github.com/OpenTTD/OpenTTD/pull/12300#pullrequestreview-1941547508
18:16:29 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1218986332295008316/image.png?ex=6609a87d&is=65f7337d&hm=13c13fc844a8e8f5d18e4cdee23fb17f0e0f59ab7265e616d02b2fa70fd10b50&
18:16:29 <andythenorth> oof I did a bad thing, but it seems to have worked
18:16:35 <andythenorth> seems pillow is looking for 0.0
18:16:50 <andythenorth> and there was already precedent for brew aliasing 0.4.0
18:27:11 <debdog> https://github.com/OpenTTD/OpenTTD/discussions/12282 "Voting opens by March 18, 2024." according to my calender we're not there yet. why has it started already?
18:32:34 <_glx_> because it's not possible to prevent things once the discussion is open
18:34:20 *** kale91 has joined #openttd
18:34:20 <kale91> https://cdn.discordapp.com/attachments/1008473233844097104/1218990818845786132/image.png?ex=6609acaa&is=65f737aa&hm=a2e5dff2628f9f59fcde37ab9ffa31547cb9f7e2d6ae092b386e8f600153f32c&
18:34:20 <kale91> is there any way we could make this error go away for forever? At least have it inform me where the mismatch is in the console
18:34:43 <kale91> incredibly frustrating trying to find the one newgrf out 200+ that it might be
18:35:02 <kale91> it gives you specific instructions for them when they're missing
18:35:26 <peter1138> There's probably a debug_level that'll tell you more.
18:35:54 <kale91> peter1138: how does that work?
18:36:27 <kale91> it just gives me grf=0
18:40:07 <peter1138> debug_level grf=1
18:40:15 <peter1138> Or viewing the gamelog might show you.
18:40:44 <kale91> peter1138: nothing happens when I input this, I will check the log
18:40:50 <kale91> but the game isnt actually running yet
18:41:01 <peter1138> Well you need that enabled and then you need to load the game.
18:41:10 <peter1138> It won't do anything retrospectively.
18:41:10 <kale91> Ah
18:41:30 <kale91> let me try it
18:41:38 <peter1138> But if it's loaded you can just type `gamelog` into the console and that'll also show things.
18:42:32 *** keikoz has quit IRC (Ping timeout: 480 seconds)
18:44:24 <kale91> alright the debug setting found it, I did have to do a lot of scrolling but it was better then guessing
18:44:31 <kale91> sorry if I was snippy, one of those days lol.
18:48:38 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #12300: Feature: encrypted game sockets https://github.com/OpenTTD/OpenTTD/pull/12300
18:55:47 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #12329: Feature: console command to change authorized keys https://github.com/OpenTTD/OpenTTD/pull/12329
18:56:38 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326
18:57:02 <truebrain> Rubidium: you resolved two comments, but they aren't fixed?
18:58:58 <Rubidium> huh?!? I must be going insane
18:59:27 <truebrain> That I cannot judge πŸ˜›
19:00:59 <Rubidium> I'm sure I made the changes... but must've gone lost because I forgot to commit or added it to the wrong thing and then rebased the change away or something
19:01:14 <truebrain> I know that pain .....
19:01:32 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326
19:01:49 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326#pullrequestreview-1941565580
19:01:58 <Rubidium> stacking just too many PRs :(
19:02:09 <truebrain> I helped unstacking them today πŸ˜„
19:02:56 <Rubidium> true, thanks for that :)
19:03:14 <truebrain> happy we finally have a solution where passwords are thrown over the wire plain-text πŸ˜›
19:03:38 <truebrain> we are getting closer to being able to store company access in the savegame πŸ˜„
19:05:47 <peter1138> Only a few years later than requested.
19:06:19 <truebrain> better late than never!
19:06:36 <Rubidium> s/years/decades/?
19:13:08 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12329: Feature: console command to change authorized keys https://github.com/OpenTTD/OpenTTD/pull/12329#pullrequestreview-1941567429
19:22:17 <peter1138> Let's not get ahead of ourselves, it's not implemented yet πŸ˜„
19:22:30 <truebrain> not with that attitude!
19:22:54 <peter1138> Somebodyβ„’ should do it.
19:23:31 <peter1138> To go with all the spyware bloat.
19:25:47 *** Leopold___ has joined #openttd
19:26:19 <truebrain> that comment was still excellent
19:32:50 *** Leopold has quit IRC (Ping timeout: 480 seconds)
19:32:56 <andythenorth> do we have much spyware yet?
19:36:28 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #12326: Feature: authorized key authentication for rcon https://github.com/OpenTTD/OpenTTD/pull/12326
19:38:08 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12329: Feature: console command to change authorized keys https://github.com/OpenTTD/OpenTTD/pull/12329
19:39:19 <Rubidium> and the console command now also allows passing the client id so you don't have to copy 64 hex characters
19:40:12 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #12329: Feature: console command to change authorized keys https://github.com/OpenTTD/OpenTTD/pull/12329#pullrequestreview-1941574025
19:40:59 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #11878: Add: public key authentication to the admin connection https://github.com/OpenTTD/OpenTTD/pull/11878
19:41:02 <truebrain> yeah, that is nice πŸ™‚
19:42:50 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #12329: Feature: console command to change authorized keys https://github.com/OpenTTD/OpenTTD/pull/12329
19:43:39 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #12329: Feature: console command to change authorized keys https://github.com/OpenTTD/OpenTTD/pull/12329#pullrequestreview-1941574649
19:44:20 <peter1138> andythenorth: Yes, all of the spyware.
19:46:18 <michi_cc> Don't you want to get rich by selling all the data out to some evil corporation? 🀣
19:47:59 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #12329: Feature: console command to change authorized keys https://github.com/OpenTTD/OpenTTD/pull/12329#pullrequestreview-1941576219
19:48:53 <truebrain> I seriously wonder if anyone would pay any money for this kind of data πŸ˜›
19:49:02 <andythenorth> can we share the proceeds with grf authors?
19:49:25 <peter1138> Based on how many variants they use?
19:49:49 <andythenorth> maybe
19:52:12 <frosch123> i use the data to train an AI to play ottd. so we can get rid of all the costly players
19:53:23 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #11878: Add: public key authentication to the admin connection https://github.com/OpenTTD/OpenTTD/pull/11878#pullrequestreview-1941576582
19:53:31 <truebrain> Sorry Rubidium, of all my nitpicks, this is the most pedantic comment of the whole weekend πŸ˜›
19:54:44 <frosch123> please don't challenge anyone on that
19:54:59 <truebrain> Add: "I made" to that sentence
19:55:00 <truebrain> pff
19:55:14 <truebrain> without it, ironically, you win the most pedantic comment of the weekend πŸ˜‰
20:05:30 *** Ox7C5 has quit IRC ()
20:21:23 *** marchnex has quit IRC (Quit: marchnex)
20:21:36 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #12329: Feature: console command to change authorized keys https://github.com/OpenTTD/OpenTTD/pull/12329
20:21:37 *** marchnex has joined #openttd
20:29:06 *** marchnex has left #openttd
20:32:19 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #11878: Add: public key authentication to the admin connection https://github.com/OpenTTD/OpenTTD/pull/11878
20:37:08 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #11878: Add: public key authentication to the admin connection https://github.com/OpenTTD/OpenTTD/pull/11878#issuecomment-2002609586
20:39:11 *** nielsm has quit IRC (Ping timeout: 480 seconds)
21:02:55 *** keikoz has joined #openttd
21:10:49 *** gelignite has quit IRC (Quit: Stay safe!)
22:13:30 *** Leopold___ has quit IRC (Ping timeout: 480 seconds)
22:15:56 *** keikoz has quit IRC (Ping timeout: 480 seconds)
22:18:43 *** Leopold has joined #openttd
22:19:48 <andythenorth> did we do vehicle permanent storage yet?
22:20:00 <andythenorth> or extensible user bits that aren't the current ones? πŸ˜›
22:20:21 <andythenorth> I have a thing where brake vans adapt to the lead engine of the consist
22:20:34 <andythenorth> but that requires checking every ID for every engine variant
22:23:20 <andythenorth> oh I could check ranges I guess, the variants use predictable increments on a base ID number
22:34:46 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
22:43:47 *** gnu_jj has quit IRC (Ping timeout: 480 seconds)
23:24:58 *** Leopold has quit IRC (Remote host closed the connection)
23:31:48 *** Leopold has joined #openttd