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

Folder placeholder objects should be set to standard storage class regardless of preference #11751

Closed
cyberduck opened this issue Aug 3, 2021 · 3 comments

Comments

@cyberduck
Copy link
Collaborator

@cyberduck cyberduck commented Aug 3, 2021

0b3458e created the issue

I am looking to use Cyberduck with client-side encryption and Amazon S3 Glacier Deep Archive. When the S3 Default Storage Class preference is set to Glacier Deep Archive, folder metadata is also stored in that way? This is a problem.

  1. Cannot upload to folders (cannot read folder meta)
  2. Can upload a folder structure to root, but cannot
    i. open it
    ii. call Restore on it (=> cannot restore any contents)

If there was another Default Storage Class preference for folders or another hardcoded default it would be very helpful. For my use case of uploading pictures, I cannot use a flat structure + multiple buckets. Additionally, the flow used to produce 2. should probably be disallowed.

For completeness, there is a workaround:

  1. Change the preference to Standard
  2. Create folder(s)
  3. Change the preference to Glacier
  4. Upload file(s)

I have marked this as a bug because it represents a break from expectations.

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Aug 4, 2021

0b3458e commented

To add to this, masterkey.cryptomator is also allowed to be a Glacier object (I think this also breaks things). It is created using Action > New Encrypted Vault.

Loading

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Sep 30, 2021

@dkocher commented

Relates to #10488.

Loading

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Oct 6, 2021

@ylangisc commented

In 6178de6.

Loading

@cyberduck cyberduck closed this Oct 6, 2021
@iterate-ch iterate-ch locked as resolved and limited conversation to collaborators Nov 27, 2021
AliveDevil pushed a commit to AliveDevil/cyberduck that referenced this issue Nov 30, 2021
…RAC-11751 to master

* commit 'eee3dcefbbc819ba45eb87cfe38c297eb4759fda':
  Always upload directory placeholders using standard storage class. Fix iterate-ch#11751.
  Always save metadata of vault using standard storage class. Fix iterate-ch#11751.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants