Skip to content
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

fix(iroh-blobs): Timeout based on correct max_write_duration option #2688

Merged
merged 2 commits into from
Sep 2, 2024

Conversation

matheus23
Copy link
Contributor

Description

This fixes a small copy and paste bug that was introduced in #2622, here: https://github.com/n0-computer/iroh/pull/2622/files?diff=unified&w=0#diff-bbe46ca272a808bb1c94619ab3264586a47a6ec6e0f7cd5a63dba5e99099241bL1521-R1526 (You need to load the diff for the fs.rs file)

Breaking Changes

None

Notes & open questions

Tests would be nice, but I'm looking at other things right now, so I'd prefer to merge this first :S

Change checklist

  • Self-review.
  • [ ] Documentation updates following the style guide, if relevant.
  • [ ] Tests if relevant.
  • All breaking changes documented.

Copy link

github-actions bot commented Sep 2, 2024

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

Last updated: 2024-09-02T12:24:33Z

@matheus23 matheus23 changed the title fix(iroh-blobs): Timeout based on correct write option fix(iroh-blobs): Timeout based on correct max_write_duration option Sep 2, 2024
@matheus23 matheus23 self-assigned this Sep 2, 2024
Copy link
Contributor

@rklaehn rklaehn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, this seems correct 😊

@matheus23 matheus23 added this pull request to the merge queue Sep 2, 2024
Merged via the queue into main with commit 2347565 Sep 2, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants