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

Allow long file names to be uploaded to S3 #167

Closed
1 task done
ptirador opened this issue Jan 5, 2021 · 0 comments · Fixed by #168
Closed
1 task done

Allow long file names to be uploaded to S3 #167

ptirador opened this issue Jan 5, 2021 · 0 comments · Fixed by #168
Milestone
1.0.0

Comments

@ptirador
Copy link
Contributor

ptirador commented Jan 5, 2021 •
edited by carlspring
Loading

Task Description

Given a file with a long filename (around 200 characters) deeper in directories it uses the whole path as a file temporary name. Unfortunately, UNIX filesystems support only files up to 255 characters, so the upload will fail with exception File name too long. S3 systems support file names up to 1024 characters as can be seen here - so there is no reason to not accept this file to upload.

Tasks

The following tasks will need to be carried out:

  • When creating a temp file in S3SeekableByteChannel, do not use the full depth of filename as a temporary name, but just the path file name.

Useful Links

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

Successfully merging a pull request may close this issue.

Issue 167: Allow long file names to be uploaded to S3 ptirador/s3fs-nio
2 participants
@carlspring @ptirador