Skip to content

feat(logger): enhance sanitize to handle Date, BigInt, and string types and add the test file.Reduce the depth from 50 to 25.#8231

Merged
rishikeshdadam136 merged 1 commit intomasterfrom
WP-8110
Mar 6, 2026
Merged

feat(logger): enhance sanitize to handle Date, BigInt, and string types and add the test file.Reduce the depth from 50 to 25.#8231
rishikeshdadam136 merged 1 commit intomasterfrom
WP-8110

Conversation

@rishikeshdadam136
Copy link
Contributor

@rishikeshdadam136 rishikeshdadam136 commented Mar 3, 2026

TICKET-8110

The sanitize() function in @bitgo/logger handles the below ones correctly and added the test files.

  • Date objects.
  • BigInt values.
  • Strings — bearer v2 tokens passed as top-level strings are not redacted.
    -Reduce the depth.

@rishikeshdadam136
Copy link
Contributor Author

@copilot Review it as 10x engineer.

Copy link

Copilot AI commented Mar 3, 2026

@rishikeshdadam136 I've opened a new pull request, #8232, to work on those changes. Once the pull request is ready, I'll request review from you.

@rishikeshdadam136
Copy link
Contributor Author

@claude review it as 10x engineer

@rishikeshdadam136 rishikeshdadam136 marked this pull request as ready for review March 3, 2026 13:46
@rishikeshdadam136 rishikeshdadam136 requested review from a team as code owners March 3, 2026 13:46
@dsobkowi dsobkowi removed the request for review from a team March 3, 2026 17:28
@sachushaji
Copy link
Contributor

@claude

@rishikeshdadam136 rishikeshdadam136 merged commit 6d786d3 into master Mar 6, 2026
31 of 32 checks passed
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.

7 participants