Closed peckover closed 2 years ago
The logging part repeats quite a bit. could be refactored to another method perhaps. otherwise LGTM!
In the connectors? I did consider it but you still have to pass most of the parameters through and it looked messy. The logger comes from the StandardDesHttpParser so you'd have to pass that through too to avoid clashing. If you've got an idea feel free to push a commit
https://github.com/hmrc/alert-config/pull/2539 <---- DO NOT MERGE THIS UNTIL MONDAY