-
Notifications
You must be signed in to change notification settings - Fork 97
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
S3 configuration and Node.js example #125
Comments
I will be updating the docs soon to use aws-sdk-ruby instead of hand rolling the signing process. The Node.js version could be simplified by doing the same. |
Yes it looks like there’s some docs for it here |
@digitaltoad Is there any update on that front? Does this project work with the format returned by the aws-sdk?
|
Right now it would not. The s3 uploader is looking for either an endpoint or a bucket name in the json response from signing. |
Sorry for the lack of updates. I have been very busy and haven't been able to update this part of the project. I'm not sure when I'll be able to get to it. As a quick fix, you could look at the s3 uploader and override the upload method. |
@digitaltoad At the moment I have a hard time getting this to work at all (following instructions or not). S3 doesn't recognize the signature. As soon as I have this working I will see if I can make a PR. Thanks for the prompt answer! |
@karellm I've had luck using the aws-sdk response directly. Your server needs to only respond with the In short, based on your example, your server's response to the signing request will look like:
It's worth noting that the Ruby aws-sdk returns it as |
@jamesfid Thanks for the feedback. I also got it working with the |
I think that the following info should be added to the wiki:
AWS Configuration
Create an access key ID for the backend using
IAM and record the Access Key ID and Secret Access Key
Add a policy to allow this user to access S3
Create a bucket on S3
Enable CORS
I've also translated the Ruby implementation into Node.js, and I think it'd be good to add this to the wiki as well:
The text was updated successfully, but these errors were encountered: