Serilog dupe stackframe fix

[jaffinito]

Thank you for submitting a pull request. Please review our contributing guidelines and code of conduct.

Description

Updates serilog wrapper to check for duplicate log entries by looking at the stackframes and checking for additional "Dispatch" methods.

There are two different versions of the stack trace, one that places the initial Dispatch method at 7 and one that places it at 8. The logic in the wrapper handles this.

Tests all pass and performance is acceptable (let me know if you want to go over it)

Closes #1070

Author Checklist

• Unit tests, Integration tests, and Unbounded tests completed
• Performance testing completed with satisfactory results (if required)
• Agent Changelog or Lambda Agent changelog updated

Reviewer Checklist

• Perform code review
• Pull request was adequately tested (new/existing tests, performance tests)
• Review Changelog

[JcolemanNR]

I think it would be vital to cover both of these stack depth use cases in integration tests so we don't introduce a regression or new bug if we refactor the current approach.

[JcolemanNR]

Thank you for adding comments! There are a few minor typos in them but I can live with that.

This needs integration test coverage still.

[vuqtran88]

We did a code walk through and addressed all of the concerned. This is good to go.

[JcolemanNR]

Great fix! Reviewed as a team and I approve.