Skip to content

Conversation

@brenuart
Copy link
Collaborator

Add a way to escape the comma in comma separated property values

Issue #847

@codecov
Copy link

codecov bot commented Oct 19, 2022

Codecov Report

Merging #882 (4771253) into main (def6f43) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #882      +/-   ##
============================================
+ Coverage     71.77%   71.83%   +0.05%     
- Complexity     1331     1334       +3     
============================================
  Files           170      170              
  Lines          4929     4939      +10     
  Branches        518      520       +2     
============================================
+ Hits           3538     3548      +10     
  Misses         1148     1148              
  Partials        243      243              
Impacted Files Coverage Δ
...in/java/net/logstash/logback/util/StringUtils.java 97.61% <100.00%> (+0.74%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@brenuart brenuart merged commit 3e8f95a into main Oct 19, 2022
@brenuart brenuart deleted the 847-value-masking-adding-multiple-values-in-a-single-values-entry-makes-the-implicit-assumption-that-the-values-to-match-does-not-contain-a-comma branch October 19, 2022 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Value Masking: adding multiple values in a single <values> entry makes the implicit assumption that the values to match does not contain a comma

3 participants