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

upload to US Standard region failing with 0.4.0 #53

Closed
astraw opened this issue Jan 19, 2015 · 1 comment · Fixed by #54
Closed

upload to US Standard region failing with 0.4.0 #53

astraw opened this issue Jan 19, 2015 · 1 comment · Fixed by #54
Labels

Comments

@astraw
Copy link

astraw commented Jan 19, 2015

Hi,

A "meteor update" broke my previously working setup in which I configure slingshot with a --settings CLI argument containing the following (substituting my actual values):

{"AWSAccessKeyId": "my_key",
 "AWSSecretAccessKey": "my_secret_key",
 "S3Bucket": "my_bucket"}

In the browser JS console, the first error I see is:

OPTIONS https://my_bucket.s3-us-east-1.amazonaws.com/ net::ERR_NAME_NOT_RESOLVED

This arises in the line xhr.send(buildFormData()); in upload.js.

And indeed, the DNS name does not resolve. However, my_bucket.s3.amazonaws.com does resolve and, as far as I can tell, this was the name slingshot 0.3.0 would use.

So it seems, at least for my use of slingshot without setting the "region" variable, the code is now broken.

@gsuess gsuess added the bug label Jan 19, 2015
@gsuess
Copy link
Contributor

gsuess commented Jan 19, 2015

Hmm, yes, totally overlooked that AWS has a discrepancy there:

Region is "us-east-1", but the domain is:

s3.amazonaws.com (N. Virginia or Pacific Northwest)
s3-external-1.amazonaws.com (N. Virginia only)

As a workaround you can set a bucketUrl to "https://my_bucket.s3.amazonaws.com" while still leaving the region at "us-east-1".

I will push a fix soon though.

astraw added a commit to strawlab/neuron-catalog that referenced this issue Jan 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants