-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Allow configuring the downloader when creating a blobs protocol handler #76
base: main
Are you sure you want to change the base?
Conversation
… handler we don't allow passing in the entire downloader, but all config options.
Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh-blobs/pr/76/docs/iroh_blobs/ Last updated: 2025-03-14T14:30:29Z |
…is properly applied. - downloader handle has the config - downloader handle gets an Arc<Inner>
# Conflicts: # src/net_protocol.rs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
semver check indicates breaking changes, should go in the PR description! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 besides me wondering about the Downloader
struct wrapping.
Description
Allow configuring the downloader when creating a blobs protocol handler
we don't allow passing in the entire downloader, but all config options.
Breaking Changes
None, just an additional fn on the builder
Notes & open questions
Change checklist