This controller all looks good, just wondering when you calling the AWS stuff like: (AwsService.postMessageToSNS('arn:aws:sns:us-west-2:005837367462:abstracts_req', abstractsChecked); should that be placed in its own service for sake of modularity and simplifiying the controller logic? So you just call the service function instead of the actual call. It won't effect the workings or performace of the code at all, it will just look sexier, but I don't mind if you do it this way either. Doesnt matter to me. Overall the controller look really cute.