-
-
Notifications
You must be signed in to change notification settings - Fork 267
Add a stackFrameFilter argument to SentryClient's capture #27
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
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. What to do if you already signed the CLAIndividual signers
Corporate signers
|
I signed it! |
This repository was supposed to be moved under https://github.com/flutter/packages but we somehow ended up in a situation where this repository was erased but the other one not populated. I'll find out what's going on. Thanks for the PR! We'll find a way to accept it, hopefully soon 😃 |
Alright, I was wondering about that :D Please tell me once you figured out where the code should go, I also realized that I have the wrong mail address for the CLA in the commit's stamp, so I'll have to replay the changes anyways. |
@tom95 The repository has been restored. Could you please rebase to kick off Travis and correct the email address for the CLA bot to recheck it? |
This allows filtering sensitive frames on the client or applying custom truncation logic.
406b667
to
750c5d6
Compare
I signed it! |
Well not entirely sure what's missing now. Mail on the commit should be correctly ending in "@4ed1.com" and it's been added to my github account as well. The corporate CLA has also been signed. |
Maybe it will help if you open a new PR with a freshly authored commit that contains the correct email? Just guessing. |
I signed it! |
CLAs look good, thanks! |
Hey, what's the status of the PR? Should it be held up by the failing travis build, I am hoping that's just because at the time of the build the CLA was not accepted, at least the log only says "presubmit" failed. |
The CLA is happy with both this PR and #30. Let's close one and merge the other? |
This change allows filtering sensitive frames on the client or applying custom truncation logic.
I created this in reaction to this discussion: https://forum.sentry.io/t/issue-in-flutter-project-with-stackframe-display-limit-set-to-250/5014/2
Instead of adding complex truncation options that will likely depend on the frameworks you use anyways, I opted to provide a more general callback that could also be used for things like filtering specific frames. In the case of above discussion, we'd simply cut the list passed to us to only send the topmost 250 items.