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.

Sign up for GitHub

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

Jump to bottom

Throw error if an entry is too long for the tar archive format used by Bee #421

Closed
Cafe137 opened this issue Oct 4, 2021 · 2 comments · Fixed by #968
Closed

Throw error if an entry is too long for the tar archive format used by Bee #421

Cafe137 opened this issue Oct 4, 2021 · 2 comments · Fixed by #968
Labels
type:issue

Comments

@Cafe137
Copy link
Collaborator

Cafe137 commented Oct 4, 2021

Swarm-CLI: ethersphere/swarm-cli#250
Bee: ethersphere/bee#2542

  1. Figure out what tar format Bee is using
  2. Check in bee-js if the data to be uploaded contains any invalid entries for that specific tar spec
  3. Throw human friendly error
@bee-runner bee-runner bot added the type:issue label Oct 4, 2021
@Cafe137 Cafe137 changed the title Throw error if an entry is too long for the tar archive format Throw error if an entry is too long for the tar archive format used by Bee Oct 4, 2021
@AuHau
Copy link
Contributor

AuHau commented Oct 6, 2021 •
edited
Loading

Well, honestly I would like to remove usage of Tar for directory upload as you can see in the #416

Unfortunately, it is currently blocked as Bee does not allow nested directory upload using multipart upload as you can see ethersphere/bee#2578

@Cafe137 Cafe137 linked a pull request Oct 24, 2024 that will close this issue
fix: throw error for too long tar entry #968
Merged
@bee-worker
Copy link
Collaborator

Release v8.3.0 addresses this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type:issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: throw error for too long tar entry ethersphere/bee-js
3 participants
@AuHau @bee-worker @Cafe137