Follow-up from "Resolve "Refactor AwsProductSender""
The following discussion from !186 (merged) should be addressed:
-
@bhawthorne started a discussion: (+1 comment) As we add this in different places thoughts about having a wrapper class around this where it can implement @Configurable and be setup like a lot of the other classes with a specific configuration as needed. It could have retry logic/better error handling built in so each class doesn't need to worry about that themselves (it could even be configurable).
It could technically be a singleton if we wanted then the HttpClient would be shared across classes utilizing it (which shouldn't be a problem because it should be able to handle that) but if each class would want its own configuration of it then we would need separate instances for it.