| 11 Aug 2025 |
Toma | * Currently the timeout logic in fetchCargoVendor is pretty bad (not even sure if it's working properly) and this tries to fix it, while also doing some general improvements | 15:35:05 |
emily | unfortunately I rebooted the VM and it fixed it 😅 | 15:35:45 |
emily | but I will try this next time | 15:35:47 |
K900 | Any reason we're not just doing async at this point | 15:35:52 |
K900 | Feels like it'll make things significantly less painful | 15:35:59 |
emily | oh wait | 15:36:04 |
emily | it worked after applying your patch and I assumed that was because I rebooted | 15:36:16 |
emily | but it fails again if I revert your patch | 15:36:20 |
emily | well, hangs again | 15:36:25 |
emily | uh | 15:36:42 |
emily | ok, but then I applied your patch again and it's back to failing | 15:36:47 |
emily | so I guess it just worked once and then I ^C'd half-way through and tried another build and it re-broke :) | 15:36:56 |
Toma | hell yeah i love consistency | 15:37:06 |
emily | I'm doubting this is a bug in your code rather than a VM/passta issue | 15:37:10 |
emily | wait no | 15:37:24 |
Toma | in any case, the timeout logic is better with my PR | 15:37:31 |