fix: Handle Exceptions and Prevent application from crashing when using Logger

[hjgraca]

Please provide the issue number

Issue number: #514

Summary

On exceptions within the Logging framework code, currently, it throws an exception and brings down the whole application.
For example below is a code that has a mismatch count of parameters passed to the log message - Missing Count property value being passed in.

Logger.LogInformation(extraKeys, "Retrieved data for city {cityName} with count {Count}", new[] { cityName });

It throws an exception and causes the Lambda Handler endpoint to crash and return a 500 Internal Server Error

System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at Microsoft.Extensions.Logging.LogValuesFormatter.GetValue(Object[] values, Int32 index)
   at Microsoft.Extensions.Logging.FormattedLogValues.get_Item(Int32 index)
   at Microsoft.Extensions.Logging.FormattedLogValues.GetEnumerator()+MoveNext(

Changes

Add exception handling in Log extension method.

try
{
    Log(logger, logLevel, extraKeys, 0, null, message, args);
}
catch (Exception e)
{
    logger.Log(LogLevel.Error, 0, e, "Powertools internal error");
}

Add Tests

User experience

The client should not see any exception in case Logger fails internally.
It will be logged as error to get visibility

Checklist

Please leave checklist items unchecked if they do not apply to your change.

• Meets tenets criteria
• I have performed a self-review of this change
• Changes have been tested
• Changes are documented
• PR title follows conventional commit semantics

Is this a breaking change?

RFC issue number:

Checklist:

• Migration process documented
• Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7308cc6) 72.23% compared to head (5159cd8) 73.32%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #538      +/-   ##
===========================================
+ Coverage    72.23%   73.32%   +1.08%     
===========================================
  Files          101      101              
  Lines         4106     4112       +6     
  Branches       418      418              
===========================================
+ Hits          2966     3015      +49     
+ Misses        1026      977      -49     
- Partials       114      120       +6     

Flag	Coverage Δ
unittests	73.32% <100.00%> (+1.08%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[amirkaws]

LGTM