Add Composer ID to some log messages

guardian / flexible-octopus-converter

An AWS Lambda for converting JSON from Octopus into Thrift.

1 stars 0 forks source link

Add Composer ID to some log messages #15

Closed jennygrahamjones closed 4 years ago

jennygrahamjones commented 4 years ago

Currently, when we log out the results of ArticleFinder.findBodyText(bundle) it can be difficult to connect the result with the corresponding Composer content. This PR adds the Composer ID to some existing log messages, to make it easier to connect any body text Articles to the relevant Bundle when investigating logs.

jennygrahamjones commented 4 years ago

👍 not for now, but if we find ourselves needing to routinely log lots of context in this app, a method like toLogMarker or similar can be handy if you've not seen it before! We use it in e.g. Grid

Yeah, you're definitely right there. Probably worth doing this sooner rather than later.