Skip to content

fix(data-masking): fail closed on masking errors - #8446

Open
leandrodamascena wants to merge 1 commit into
developfrom
fix/data-masking-fail-closed
Open

fix(data-masking): fail closed on masking errors#8446
leandrodamascena wants to merge 1 commit into
developfrom
fix/data-masking-fail-closed

Conversation

@leandrodamascena

Copy link
Copy Markdown
Contributor

Issue number: closes #8445

Summary

fail closed on masking errors

Changes

Please provide a summary of what's being changed

User experience

Please share what the user experience looks like before and after this change


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.

@leandrodamascena
leandrodamascena requested a review from a team as a code owner September 8, 2026 11:49
@powertools-for-aws-oss-automation powertools-for-aws-oss-automation Bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 8, 2026
@powertools-for-aws-oss-automation

Copy link
Copy Markdown

Referenced issues are closed or don't exist. Please ensure there is an open issue related to this change to avoid significant delays or closure.

@sonarqubecloud

sonarqubecloud Bot commented Sep 8, 2026

Copy link
Copy Markdown

@codecov

codecov Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.64%. Comparing base (a39e101) to head (3a7e649).
⚠️ Report is 5 commits behind head on develop.

Files with missing lines Patch % Lines
...s_lambda_powertools/utilities/data_masking/base.py 85.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8446      +/-   ##
===========================================
- Coverage    96.65%   96.64%   -0.02%     
===========================================
  Files          296      296              
  Lines        14767    14771       +4     
  Branches      1246     1247       +1     
===========================================
+ Hits         14273    14275       +2     
- Misses         359      361       +2     
  Partials       135      135              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mergify

mergify Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

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

Labels

size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants