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

FIX: wrong key level in object-storage config in filesystem.md #1696

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

GerDner
Copy link
Contributor

@GerDner GerDner commented Feb 13, 2025

"options:" does not exist anymore in config and was not removed here but later in the docs. This will result in an "access denied" error as soon as file system calls are made, despite valid credentials.
It seems "visibility:" was also moved one level higher as mentioned later in the docs.

"options:" does not exist anymore in config and was not removed here but later in the docs. This will result in an "access denied" error as soon as file system calls are made, despite valid credentials.
@Isengo1989 Isengo1989 added the Core @ct-core label Feb 13, 2025
@Isengo1989
Copy link
Collaborator

Seems like it exists still, but passing visibility and cred changed

image

Can you please double check @janbuecker

@janbuecker
Copy link
Member

@shyim added the visibility option long time ago. Do you know if it's still correct?

@janbuecker janbuecker requested review from shyim and removed request for janbuecker February 13, 2025 15:08
@GerDner
Copy link
Contributor Author

GerDner commented Feb 13, 2025

Just to be clear:

https://developer.shopware.com/docs/guides/hosting/infrastructure/filesystem.html#amazon-s3

works:

https://developer.shopware.com/docs/guides/hosting/infrastructure/filesystem.html#fallback-adapter-configuration

does not work.

As visibility also was moved in the example above, I assumed options were dropped completely. But i did not observe issues with visibility. Only Creds.

Edit: mixed up the links

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core @ct-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants