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

Adds access & destination config options for grunt-aws #66

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

moladukes
Copy link

@moladukes moladukes commented Oct 13, 2021

  1. While setting up s3 I was getting Access Denied errors which were solved by passing an Access permission option.

access (String)
Default "public-read"

File permissions, must be one of:

"private"
"public-read"
"public-read-write"
"authenticated-read"
"bucket-owner-read"
"bucket-owner-full-control"
  1. Once passing the correct access permission (in my case private) the upload worked! However a new problem arose; They were getting dumped into root.

grunt-aws has a dest (String) option which allows you to specify where to upload the files. By including this option my use case/set up was complete.

      build: {
        cwd: path.images_src,
        src: '**',
        dest: "<%= config.s3.dest %>"
      }
  1. Another potential "Fix" would simply add these edge cases to the Wiki however I think them common enough for them to be in the example/default configs.

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

Successfully merging this pull request may close these issues.

1 participant