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

Loosen aws-sdk version #74

Merged
merged 1 commit into from
Nov 2, 2016

Conversation

ryansch
Copy link
Contributor

@ryansch ryansch commented Nov 1, 2016

What Changed & Why

The current aws-sdk version string ~2.3.19 forces an old version of aws-sdk due to brokenness in 2.4.0. The signing issues mentioned #59 in seem to have been fixed in 2.4.1.

Personally I'm blocked on the ECS Credential stuff added in 2.4.7. 2.6.12 is current newest release.

Related issues

#59
aws/aws-sdk-js@v2.4.0...v2.4.1

@ghedamat
Copy link
Contributor

ghedamat commented Nov 1, 2016

@ryansch can you confirm that an end to end deploy with this update works? (we sadly don't have an end to end integration test)

also, would it be more prudent to lock to ~2.6.12 instead of ^ to avoid problems like last time?

@ryansch
Copy link
Contributor Author

ryansch commented Nov 1, 2016

I'll fork and confirm.

The problem is that when aws releases 2.7.x we'll be stuck on 2.6.x if we use ~2.6.12. The problem here was one bad release. We should have just blacklisted 2.4.0 (ex: ~2.3.19 || ^2.4.1).

@ghedamat
Copy link
Contributor

ghedamat commented Nov 1, 2016

fair enough,

if your smoke test passes and you can use this locally I think I'm ok in merging then!

@lukemelia
Copy link
Contributor

My semver is not excellent, but can't we use ~2.6 to get the latest 2.x?

@lukemelia
Copy link
Contributor

Scratch that suggestion, doesn't do what I thought it did per https://semver.npmjs.com/

@ghedamat
Copy link
Contributor

ghedamat commented Nov 1, 2016

@lukemelia the code in this PR that uses ^ will do what both you and @ryansch are looking for, I was just being overly prudent after getting burned last time but this is probably fine :)

@ryansch
Copy link
Contributor Author

ryansch commented Nov 2, 2016

@lukemelia lukemelia merged commit c9adf1e into ember-cli-deploy:master Nov 2, 2016
@lukemelia
Copy link
Contributor

Thanks @ryansch. Released as 0.4.0

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.

3 participants