[Feature] Add validation on dictionary value

[kzanetty]

Status

READY

Whats?

Add validation to ensure that the value is not null. If it is, the field should be completely masked.

Why?

Currently, if null is passed as the value, JsonMasking is aborted and AspNet.Serilog returns the object unchanged, exposing all data. This results in logs containing open card data and other sensitive information.

How?

This can be resolved by adding a simple validation. If the delegate is null, it should completely mask the data instead of aborting the JsonMasking.

Attachments (if appropriate)

Definition of Done:

• [ ] Increases API documentation
• [ ] Implements integration tests
• [X] Implements unit tests
• [ ] Is there appropriate logging included?
• [ ] Does this add new dependencies?
• [ ] Does need add new version in changelog?
• [ ] Does need update readme, contributing, etc?
• [ ] Does need change in CI server?
• [ ] Will this feature require a new piece of infrastructure be implemented?
• [ ] Does this PR require a blog post? If so, ensure marketing has signed off on content.

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (d8831e2) to head (c0e90cd).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #10 +/- ## =========================================== + Coverage 93.84% 100.00% +6.15% =========================================== Files 1 1 Lines 65 65 Branches 19 20 +1 =========================================== + Hits 61 65 +4 + Misses 3 0 -3 + Partials 1 0 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.