-
Notifications
You must be signed in to change notification settings - Fork 104
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
Update API version for ItemLookup to 2013-08-01 #35
Conversation
Changed API version from 2011-08-01 to 2013-08-01 when using the ItemLookup method.
Great! Thanks @FdezRomero |
Update API version for ItemLookup to 2013-08-01
Awesome! Thanks @t3chnoboy |
@masterT @JuanCrg90 Can you guys please publish a new version? |
All right 🚢! |
@t3chnoboy @FdezRomero Something is broken! |
@t3chnoboy @FdezRomero First time the specs were run by Travis CI, some did not passed. Amazon returns an unexpected error because there were to much request sent in a too short time. I will update the specs now so it won't happened again. |
Here is the Amazon error: $: { xmlns: 'http://ecs.amazonaws.com/doc/2013-08-01/' },
Error: [
{
Code: [ 'RequestThrottled' ],
Message: [
'AWS Access Key ID: AKIAIXILNIR67IOXLNHA. You are submitting requests too quickly. Please retry your requests at a slower rate.'
]
}
],
RequestId: [ '6a410764-3977-4149-bdfd-383893c15992' ]
} |
Same here, |
This is because you have a maximum rate of 1 request per seconde per IP address. You need to throttle your request. @arjunkori check the issue #37. |
Changed API version from 2011-08-01 to 2013-08-01 when using the ItemLookup method as discussed in #26. All tests are passing OK.
You may want to update the dependencies (the badge is showing yellow at the moment) and bump the package version to NPM afterwards.
Thanks!