n0-computer / iroh-blobs

Blobs layer for iroh
Apache License 2.0
4 stars 0 forks source link

feat: `import_verifiable_stream` and `write_verifiable_stream` in `Store` #10

Open matheus23 opened 23 hours ago

matheus23 commented 23 hours ago

Description

These allow interacting with the iroh_blobs store using iroh's bao-style streams.


I'm coming across this while extracting iroh-willow into its own thing. This means merging in all work that's not yet merged into iroh yet from the willow PR/branch in iroh. This is one of these things.

IIUC, this allows us to stream the willow entry payloads directly from the WGPS protocol instead of having to simultaneously run iroh-blobs & iroh-willow.

Breaking Changes

Adding fn to traits should be non-breaking as long as they have default impls, right?

Notes & Open Questions

Unfortunately no tests with this yet, other than my pinky-promise that this code has worked so far for iroh-willow. That said, I haven't tried bigger payloads yet.

I'll probably make use of this branch for the iroh-willow extraction and come back eventually to add proper tests.

Change checklist

matheus23 commented 23 hours ago

I still requested a review just to confirm: Is this a feature we want to support in iroh-blobs? If so, I'll come back and add some tests to this. If not, I won't waste time on writing tests & we'll need to think of some other way of doing this & revise the use in iroh-willow.

github-actions[bot] commented 23 hours ago

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh-blobs/pr/10/docs/iroh_blobs/

Last updated: 2024-11-13T18:34:17Z