Skip to content

Conversation

@thomashmolina
Copy link
Contributor

@thomashmolina thomashmolina commented Oct 4, 2024

Description

Related to this issue. This enables redacted values of readonly properties to appear in logs.

Added 2 additional tests to verify that readonly properties can be redacted.

Expected Behavior

Logs should show redacted values of readonly properties if they defined in the $sensitive_keys array.

Fix

traverseObj checks if an object has readonly properties. If an object has readonly properties, it uses an instance of \stdClass to record redacted versions of the value instead of trying to modify a readonly property.

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.

1 participant