Skip to content

LogMasked works for properties of any type#141

Merged
sungam3r merged 1 commit into
masterfrom
string
Dec 20, 2024
Merged

LogMasked works for properties of any type#141
sungam3r merged 1 commit into
masterfrom
string

Conversation

@sungam3r
Copy link
Copy Markdown
Member

fixes #139

@sungam3r sungam3r added the enhancement New feature or request label Dec 18, 2024
@github-actions github-actions Bot added documentation Improvements or additions to documentation tests Pull request that adds new or changes existing tests labels Dec 18, 2024
var sv = (StructureValue)evt.Properties["Customized"];
var props = sv.Properties.ToDictionary(p => p.Name, p => p.Value);

props.ContainsKey("ShowFirstFiveThenCustomMaskEnum").ShouldBeTrue();

Check notice

Code scanning / CodeQL

Inefficient use of ContainsKey

Inefficient use of 'ContainsKey' and [indexer](1).
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (2675b66) to head (90a7d1e).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #141   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           11        11           
  Lines          245       242    -3     
  Branches        37        37           
=========================================
- Hits           245       242    -3     

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request tests Pull request that adds new or changes existing tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Masking Not Applied for Enum and double Properties in Logs

2 participants