Skip to content
This repository has been archived by the owner on Nov 24, 2021. It is now read-only.

ACL: "public-read" #36

Closed
g3r4n opened this issue Jun 18, 2018 · 9 comments
Closed

ACL: "public-read" #36

g3r4n opened this issue Jun 18, 2018 · 9 comments

Comments

@g3r4n
Copy link

g3r4n commented Jun 18, 2018

ACL in uploadParams don't seems to work.
I've tried with keystone-storage-adapter-s3 v2.0.0

@g3r4n
Copy link
Author

g3r4n commented Jun 18, 2018

got only the issue for file with a spaces in the name.

@mikehazell
Copy link
Contributor

@g3r4n - This is working for me. Can you post your adapter config?

@g3r4n
Copy link
Author

g3r4n commented Jun 18, 2018

@mikehazell Yes i can confirm the issue comes from files with spaces in the name. i used the generateFilename option to fixe it.
By the way, this option is not documented.

@g3r4n g3r4n closed this as completed Jun 18, 2018
@mikehazell
Copy link
Contributor

Ok thanks @g3r4n. Were you seeing this issue with the default generateFileName option (ie. random filename) or were you using something custom?

@g3r4n
Copy link
Author

g3r4n commented Jun 18, 2018

@mikehazell It was with a custom generateFileName function. Maybe we can improve the code by flaging something in the console ?
Thanks for your work and your quick replies :)

@mikehazell
Copy link
Contributor

Yeah, a console warning might be helpful. Would be interesting to know if other characters cause this issue. Thanks for the heads up.

@jennaprice
Copy link

I am having this issue as well. I am getting with devops to see if its a permissions error, but I cannot upload images if I use the upload params

@g3r4n
Copy link
Author

g3r4n commented Jul 4, 2018

@jennaprice did you try to upload a file without space in the name ?

@jennaprice
Copy link

@g3r4n Hmm I just saw this, it turned out to be a permission issue from DevOps after a ton of troubleshooting.

@g3r4n g3r4n closed this as completed Sep 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants