germanattanasio / visual-recognition-nodejs

DEPRECATED: this repo is no longer actively maintained
Apache License 2.0
362 stars 351 forks source link

security.js - for VR Nodejs Rate limiting to API is set too low at 10 should be higher. #255

Closed ghost closed 7 years ago

ghost commented 7 years ago

A customer using the VR Nodejs demo was getting 429 errors after uploading more than 10 images. this number is set too low for most customer demos and evaluations and should be set to a higher number to reduce the occurrence of hitting 429 errors.

germanattanasio commented 7 years ago

10 images every 30 seconds seems like a good rate limitation for a service demo. The customer can always download the code and create it's own service instance and that won't have the rate limitation