Skip to content

Conversation

@cl-efornaciari
Copy link
Contributor

@cl-efornaciari cl-efornaciari commented Dec 4, 2025

We found some fields marked as sensitive not properly being redacted from the /status page due to newline characters. This change introduces logic to handle this case

@cl-efornaciari cl-efornaciari requested a review from a team as a code owner December 4, 2025 22:49
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

👋 cl-efornaciari, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

NPM Publishing labels 🏷️

🟢 This PR has valid version labels and will cause a patch bump.

// Escaping potential special characters in values before creating regex
.replace(/[-[\]{}()*+?.,\\^$|#\s]/g, '\\$&')
// Escaping special case for new line characters. This is needed to properly match and censor private keys,
// ssh keys, and other multi-line string values.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ty!

@cl-efornaciari cl-efornaciari force-pushed the feature/DF-22512/handle-private-keys branch from d7b02dc to d351ac1 Compare December 8, 2025 18:58
@mxiao-cll mxiao-cll merged commit 9e39280 into main Dec 8, 2025
14 checks passed
@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

🚀 Successfully created version bump PR: #641

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants