Skip to content
This repository has been archived by the owner on Mar 26, 2024. It is now read-only.

s3:// box urls no longer work with vagrant 1.8.1 #27

Closed
nichenke opened this issue Jan 12, 2016 · 3 comments
Closed

s3:// box urls no longer work with vagrant 1.8.1 #27

nichenke opened this issue Jan 12, 2016 · 3 comments

Comments

@nichenke
Copy link

After upgrading to Vagrant 1.8.1, our private metadata/boxes stopped working.

$ vagrant box add --name versity/vsm-devtest s3://vsm-dev/vbox/vsm-devtest
==> box: Box file was not detected as metadata. Adding it directly...
==> box: Adding box 'versity/vsm-devtest' (v0) for provider: virtualbox
box: Downloading: s3://vsm-dev/vbox/vsm-devtest

If we change the url to the full https, it starts working again:

$ vagrant box add https://s3-us-west-2.amazonaws.com/vsm-dev/vbox/vsm-devtest
==> box: Loading metadata for box 'https://s3-us-west-2.amazonaws.com/vsm-dev/vbox/vsm-devtest'
box: URL: https://vsm-dev.s3-us-west-2.amazonaws.com/vbox/vsm-devtest?X...
==> box: Adding box 'versity/vsm-devtest' (v1.2.5) for provider: virtualbox
box: Downloading: s3://vsm-dev/vbox/devtest/Versity-info-virtualbox.box

It looks like something is missing the s3:// url munging. This change to https:// urls seems to work for us with Vagrant 1.7.4 as well, so we are going to push Vagrantfile changes to work around this for now.
@benesch
Copy link
Contributor

benesch commented Jan 14, 2016

Urgh. That's what I get for relying on private API.

Laptop's in the shop until next week, but this in this month's queue. Unless, of course, you find some bandwidth to submit a PR!

@benesch
Copy link
Contributor

benesch commented Jan 18, 2016

Agh. hashicorp/vagrant@891c47c

benesch added a commit that referenced this issue Jan 18, 2016
@benesch
expand S3 shorthand urls in authenticate_box_url hook
608e263 
Vagrant 1.8 only supports HTTP metadata boxes. Expand s3:// shorthand
URLs in the authenticate_box_url hook so that Vagrant's metadata check
sees HTTP URLs. This method should also be compatibile with Vagrant
1.5+.

Fix #27.
benesch added a commit that referenced this issue Jan 19, 2016
@benesch
expand S3 shorthand urls in authenticate_box_url hook
84ec86a 
Vagrant 1.8 only supports HTTP metadata boxes. Expand s3:// shorthand
URLs in the authenticate_box_url hook so that Vagrant's metadata check
sees HTTP URLs. This method should also be compatibile with Vagrant
1.5+.

Fix #27.
@benesch
Copy link
Contributor

benesch commented Jan 19, 2016

Fix released in v1.3.0. Thanks for reporting!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@benesch @nichenke