8 Oct 2025 |
John Ericson | fwiw at Nix Con I thought Jane Street people mentioned they needed auth for ldap too | 15:46:56 |
John Ericson | so that is a second use-case right there | 15:47:11 |
John Ericson | lovesegfault: if you do manage to make my acceptions go, I would PR username auth first, before the current PR, if it is not to hard to write an integration test which tests it | 15:50:16 |
John Ericson | * lovesegfault: if you do manage to make my suggested changes work, I would PR username auth first, before the current PR, if it is not to hard to write an integration test which tests it | 15:50:41 |
John Ericson | (but also can wait for someone else, e.g. Sergei Zimmerman (xokdvium) to weigh in first, and establish it's not just me thinking these things :)) | 15:51:29 |
lovesegfault | i don't see how I could possibly test it? | 18:33:59 |
lovesegfault | unless you want me to expose username/password in the store config? | 18:34:11 |
lovesegfault | John Ericson: here you go https://github.com/NixOS/nix/pull/14190 | 18:40:48 |
John Ericson | (sorry just saw, will be in meeting today) | 18:41:13 |
John Ericson | will look at PR | 18:41:30 |
John Ericson | we have done _NIX_TEST env var hacks to help with integration testing before | 18:41:40 |
John Ericson | so we could set admin:their-password that way or somthing, just spit-balling | 18:42:06 |
lovesegfault | i mean, it will get tested as the curl-s3 work lands | 19:04:13 |
lovesegfault | and if it's not working, i'll be forced to fix it to make the s3 tests pass | 19:04:27 |
lovesegfault | so i'd argue that it lands even without tests right now as it will get tests in the next couple carve-outs | 19:04:54 |
John Ericson | Redacted or Malformed Event | 19:59:10 |
| tavinator joined the room. | 22:36:39 |
9 Oct 2025 |
| chipre joined the room. | 01:39:53 |
lovesegfault | next bit of the s3 work: https://github.com/NixOS/nix/pull/14198 | 02:45:49 |
lovesegfault | we're getting close! | 02:46:07 |
lovesegfault | once the above land, next steps:
- all builtin fetchurl pre-resolution stuff
- add the s3 test suite
- switch the default
- remove the legacy impl
| 02:59:24 |
lovesegfault | * once the above lands, next steps:
- all builtin fetchurl pre-resolution stuff
- add the s3 test suite
- switch the default
- remove the legacy impl
| 04:06:45 |
lovesegfault | Sergei Zimmerman (xokdvium): https://github.com/NixOS/nix/pull/13752/commits/54e39e890bee16a1062bf33314d32a8cf20ab870
what do you think?
| 05:22:34 |
| @nullcube:matrix.org left the room. | 10:36:17 |
fzakaria | i was traveling yesterday | 16:58:33 |
fzakaria | what happened with all the fuzzing stuf | 16:58:37 |
lovesegfault | it | 17:14:59 |
lovesegfault | * it's there, i don't have time to push it across the finish line right now | 17:15:10 |
lovesegfault | it found some crashes | 17:15:22 |
| @notgne2:wizbos.club left the room. | 20:19:19 |