Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Enhanced variable scrubbing and context vars #37

Merged
merged 2 commits into from
Oct 25, 2024

Conversation

stephanschuler
Copy link
Member

Scrub frame variables manually in case scrubbing isn't already done by zend.exception_ignore_args.

This allows for individual per-frame decissions to either drop certain variables or keep them. It doesn't change, however, how frame variables are transported to Sentry, which usually is very none-descriptive due to how Sentry handles variable repreesntation.

To add more details to variables, specific frame variables can be analyzed and copied from frames to the event context befor being potentially scrubbed and stringified.

Scrub frame variables manually in case scrubbing isn't already
done by zend.exception_ignore_args.

This allows for individual per-frame decissions to either drop
certain variables or keep them. It doesn't change, however, how
frame variables are transported to Sentry, which usually is
very none-descriptive due to how Sentry handles variable
repreesntation.

To add more details to variables, specific frame variables
can be analyzed and copied from frames to the event context
befor being potentially scrubbed and stringified.
@stephanschuler stephanschuler added this pull request to the merge queue Oct 25, 2024
Merged via the queue into master with commit 5bc037c Oct 25, 2024
20 checks passed
@paxuclus paxuclus deleted the feature/frame-scrubbing-and-context-variables branch October 25, 2024 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants