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

Update go-unixfsnode to 1.8.0 #10141

Merged
merged 2 commits into from
Sep 25, 2023
Merged

Update go-unixfsnode to 1.8.0 #10141

merged 2 commits into from
Sep 25, 2023

Conversation

bmwiedemann
Copy link
Contributor

@bmwiedemann bmwiedemann commented Sep 22, 2023

Fixes #9044 = regression introduced in 0.13.0 of FUSE access to some files

@bmwiedemann bmwiedemann requested a review from a team as a code owner September 22, 2023 12:56
@bmwiedemann bmwiedemann force-pushed the fuse-issue9044 branch 2 times, most recently from da236db to 0e2d64d Compare September 22, 2023 13:09
Fixes ipfs#9044 regression in 0.13.0 FUSE access to some files
docs/changelogs/v0.23.md Outdated Show resolved Hide resolved
Copy link
Contributor

@Jorropo Jorropo left a comment

Choose a reason for hiding this comment

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

Thx a lot for handling this, the debugging you done is awesome, the ultimate way to properly fix this is to add tests too so we can catch theses regressions earlier (self targeted comment, not asking you to write more code).
I've tweaked a bit the error message.

@Jorropo Jorropo merged commit 0f1ec59 into ipfs:master Sep 25, 2023
19 checks passed
@bmwiedemann bmwiedemann deleted the fuse-issue9044 branch September 25, 2023 18:15
@bmwiedemann
Copy link
Contributor Author

You mentioned directory listings in the changelog, but the trouble was (also?) about accessing files >256k that did not fit into one raw block and thus used a protobuf file entry. e.g. /ipfs/bafybeighf5coeur6nejmyjzdatfcs6ohpo6ea5lhvhmrfecy6lvpird34m

@Jorropo
Copy link
Contributor

Jorropo commented Sep 26, 2023

Updated thx #10146

@lidel lidel mentioned this pull request Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

fuse pb regression: could not convert protobuf or raw node
2 participants