-
-
Notifications
You must be signed in to change notification settings - Fork 87
Fix tag in digital ocean spaces #138
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 tag in digital ocean spaces #138
Conversation
I'm not sure we need an additional |
@mtrezza Right, I will only try to make a condition if it is empty the tags do not send to digital ocean spaces, what do you think? |
The problem is that digital ocean spaces don't accept tags anymore, so if they put tags in the files, it will give an error in the same way |
Please see the discussion in #135. The idea is that if tag is:
We want to stay true to the value the developer sets in the file upload. Do you see a change necessary in S3 adapter, in Parse Server or both for this? |
In this case I believe that the best would be only on the s3 adapter Because in the documentation of digital ocean spaces, there is no longer to send tags, so even if I send an empty object, it will give an error https://developers.digitalocean.com/documentation/spaces/#object |
I will close this PR, because as there is already another open resolving this, in the meantime I will be using my fork to solve my problem |
What I meant was that we want to give developers both options:
Otherwise it would mean that we change the S3 adapter because a 3rd party does not emulate the AWS S3 endpoint correctly, which seems counter-intuitive, because we want to design the S3 adapter according to AWS S3, which is the authoritative provider. |
Fixes #137