| 21 May 2024 |
delroth | got some time this evening to write down my current S3 plans and before filing an issue to track the work I'd like to get some consensus on whether that's the right direction :) | 15:58:59 |
puck | a pad, probably | 16:01:21 |
jade_ | granted pad access to the private beta group. we need to do something better about this but for now you will have access. | 16:30:38 |
K900 | Read that as "paid access" | 16:37:39 |
delroth | In reply to @jade_:matrix.org granted pad access to the private beta group. we need to do something better about this but for now you will have access. do you have a URL for that? | 16:43:19 |
Qyriad | https://pad.lix.systems | 16:57:52 |
| ajs124 joined the room. | 18:09:05 |
delroth | https://pad.lix.systems/s/k6jQQtEF6 quick dump | 19:36:12 |
delroth | hmm pad.lix.systems URLs don't seem to be accessible for logged out users | 19:36:36 |
delroth | not great | 19:36:39 |
puck | adjust it | 19:36:48 |
puck | press the "limited" in the top right of the (visual) editor | 19:36:59 |
puck | also, just share the pad url, rather than the "publish" link | 19:37:07 |
delroth | * https://pad.lix.systems/MP-uY5PtQI-_LDhI6Dtw_A quick dump | 19:37:44 |
delroth | done, thanks | 19:37:47 |
puck | fwiw, HttpBinaryCacheStore already implements upload | 19:38:35 |
delroth | oh, how did I miss that, I thought I checked | 19:39:34 |
delroth | ah because the virtual method is upsertFile not uploadFile (the S3 backend has both methods, but the latter is just an internal helper) | 19:40:20 |
puck | delroth also you missed that 272 suggests just vendoring the part of the aws sdk that does profile handling; rather than nuking it as such; iirc artemist was one of the people using the credential helper stuff from the aws sdk | 19:43:33 |
puck | which is a bit more of an overkill solution, but | 19:44:09 |
delroth | I think for the same reason that reimplementing it would be tricky, I expect vendoring it to require a lot of detangling | 19:44:22 |
delroth | there's likely a bunch of internal deps on util libs for stuff like API queries to IMDS | 19:44:37 |
puck | yeah, exactly | 19:45:05 |
delroth | similar to why switching to the credentials loading part of that Rust lib would probably be a bad idea - it depends on Hyper | 19:45:06 |
delroth | also: grep -r s3: tests/ returns no results 🤡 | 19:45:37 |
delroth | so I added an item for that | 19:45:44 |
delroth | I'm sure there's a minimalistic enough S3 implementation we can use to test stuff against | 19:46:54 |
delroth | (at least in integration tests) | 19:47:00 |
K900 | minio? | 19:47:03 |
puck | i mean, generating the needed HMAC signatures isn't hard | 19:47:23 |