| 31 Mar 2023 |
K900 | And there's always an issue about PAM on the first page | 16:52:35 |
K900 | And it's always a different one | 16:52:39 |
K900 | Today's is https://github.com/microsoft/WSL/issues/9881 | 16:52:42 |
Arson Copperflame | It took me literal weeks to get PAM working (well enough) with syschdemd. I don't envy them, lol. PAM is a dumpster fire of a software | 16:54:37 |
K900 | PAM isn't too bad if you use it correctly tbh | 16:55:21 |
K900 | It's just that syschdemd does things that absolutely do not fall under "using it correctly" | 16:55:38 |
Arson Copperflame | "Open a shell that runs in a PAM session" sounds like something that should be a supported usecase, tbh. The problem is that as far as I understood that, the only way to do that is a C library | 16:59:32 |
Arson Copperflame | So we just abuse runuser for that now | 16:59:49 |
K900 | Well usually whatever is starting the session is also what's handling auth and such | 17:02:40 |
K900 | Starting a session from an already authenticated session is not a common use case | 17:03:22 |
Arson Copperflame | Hot take: "It's handling auth" and "It's written in C" are two things that shouldn't occur in a program at the same time. At least not in 2023. You can pick one of those, but both is dangerous | 17:04:34 |
K900 | Too late now | 17:05:04 |
K900 | I mean, I 100% agree, but PAM is ancient | 17:05:25 |
| 1 Apr 2023 |
purewaterisme | Redacted or Malformed Event | 23:54:21 |
| 2 Apr 2023 |
| cycyone3 changed their profile picture. | 03:02:33 |
| cycyone3 changed their profile picture. | 04:36:43 |
Arson Copperflame | In reply to @nzbr:nzbr.de https://github.com/nix-community/NixOS-WSL/issues/238 I added a "Fix" (but I wouldn't bet on it working indefinitely) | 08:35:40 |
K900 | You can also fix it by wrapping ~/.vscode-server/$hash/node | 08:39:54 |
K900 | But that's a pain to automate | 08:40:01 |
Arson Copperflame | Isn't that what the nixos-vscode-server thing does though? | 08:42:40 |
K900 | Kinda | 08:43:43 |
K900 | It also tries to patchelf it | 08:43:48 |
K900 | Which isn't necessary if you have nix-ld | 08:43:49 |
purewaterisme | In reply to @nzbr:nzbr.de I added a "Fix" (but I wouldn't bet on it working indefinitely) Oh yeah that was me: I didn't even need to run that command. All I did was downgrade the extension to 0.76.0 in the GUI(which that command should be equivalent to) | 08:45:23 |
purewaterisme | In reply to @nzbr:nzbr.de I added a "Fix" (but I wouldn't bet on it working indefinitely) * Oh yeah that was me: I didn't even need to run that command. All I did was downgrade the extension to 0.76.1 in the GUI(which that command should be equivalent to) | 08:46:34 |
Arson Copperflame | Good to know, I changed the text | 08:48:10 |
purewaterisme | In reply to @k900:0upti.me Which isn't necessary if you have nix-ld I thought the whole issue was that the environment variables wouldn't be pulled and the server-env script wouldn't run with the removal of that option | 08:48:55 |
K900 | The server-env script doesn't run because it tries to spawn a relay process first | 08:49:47 |
K900 | And that relay process uses their prebuilt Node | 08:49:55 |
K900 | And that prebuilt Node doesn't run on unpatched NixOS for obvious reasons | 08:50:07 |