Skip to content

Conversation

ramya18101
Copy link
Contributor

@ramya18101 ramya18101 commented Jul 21, 2025

🔧 Changes

Add support for new field pii_config in logStreams

📚 References

🔬 Testing

Update testcases

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@ramya18101 ramya18101 changed the title Support PII Config in logStreams [DO NOT MERGE] Support PII Config in logStreams Jul 21, 2025
@ramya18101
Copy link
Contributor Author

❗️❗️❗️ Dependant on node-auth0 release for the support of pii_config

@codecov-commenter
Copy link

codecov-commenter commented Jul 21, 2025

Codecov Report

❌ Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.32%. Comparing base (87bcb42) to head (b6ab41a).

Files with missing lines Patch % Lines
src/tools/auth0/handlers/logStreams.ts 66.66% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1130      +/-   ##
==========================================
- Coverage   80.32%   80.32%   -0.01%     
==========================================
  Files         131      131              
  Lines        4981     4980       -1     
  Branches      983      983              
==========================================
- Hits         4001     4000       -1     
  Misses        564      564              
  Partials      416      416              

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ramya18101 ramya18101 changed the title [DO NOT MERGE] Support PII Config in logStreams Support PII Config in logStreams Aug 4, 2025
@ramya18101 ramya18101 marked this pull request as ready for review August 4, 2025 09:08
@ramya18101 ramya18101 requested a review from a team as a code owner August 4, 2025 09:08
@ramya18101 ramya18101 merged commit 578148b into master Aug 4, 2025
8 checks passed
@ramya18101 ramya18101 deleted the DXCDT-1046-support-pii-config-in-logStreams branch August 4, 2025 11:42
@kushalshit27 kushalshit27 mentioned this pull request Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants