Timestamper improvements (perf, potential bug fix)

Arran Walkerrequested to merge
ajwalker/timestamper-fixes into main

What does this MR do?

  • Make timestamper Logger.Close idempotent
  • Improve performance

Why was this MR needed?

This package I'll soon be copying over to step-runner, so that it can handle its side of the timestamping, so it felt worth revisiting to ensure it's correct, performant and has decent test coverage.

What's the best way to test this MR?

Existing test coverage + new tests ought to be enough.

What are the relevant issue numbers?

Merge request reports