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

Remove debugging functions from public API #468

Merged

Conversation

kattrali
Copy link
Contributor

Short and sweet. removing without replacement, as these weren't intended for public use.

setSuspendThreadsForUserReported will be resurrected in a more useful form as a configuration option in a later changeset.

@kattrali kattrali force-pushed the kattrali/remove-debugging-fns-from-public-api branch from 82d5834 to aad050f Compare February 19, 2020 16:56
@kattrali kattrali force-pushed the kattrali/remove-debugging-fns-from-public-api branch from aad050f to e3d6fbb Compare February 25, 2020 10:02
@robinmacharg robinmacharg self-requested a review February 26, 2020 12:47
Copy link
Contributor

@robinmacharg robinmacharg left a comment

Choose a reason for hiding this comment

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

LGTM!

@kattrali kattrali merged commit 1fe443b into spec-compliance Feb 26, 2020
@kattrali kattrali deleted the kattrali/remove-debugging-fns-from-public-api branch February 26, 2020 14:07
@zhaoze1991
Copy link

zhaoze1991 commented Jun 22, 2020

hi @kattrali @robinmacharg , I was working on upgrade bugsnag from v5 to v6...
our app has huge perf issue when reporting user errors so I used setSuspendThreadsForUserReported:NO to solve the issue.

with this pull request I can't call this API any more. what's the alternative way to disable suspendThreadsForUserReported?

Thanks!

@twometresteve
Copy link
Contributor

hi @kattrali @robinmacharg , I was working on upgrade bugsnag from v5 to v6...
our app has huge perf issue when reporting user errors so I used setSuspendThreadsForUserReported:NO to solve the issue.

with this pull request I can't call this API any more. what's the alternative way to disable suspendThreadsForUserReported?

Thanks!

Hi @zhaoze1991 ,

I believe you want the sendThreads configuration option:
https://docs.bugsnag.com/platforms/ios/configuration-options/#sendthreads

Thanks,
Steve.

@zhaoze1991
Copy link

hi, @twometresteve thanks for this, set sendThreads can workaround the issue I have.

But shouldn't suspendThreadsForUserReported be disabled? it's scary reporting user error could suspend all the threads.

@zhaoze1991
Copy link

hi, @twometresteve after playaround with sendThreads, this is still a perf issue for us if we have multiple errors happened simultaneously .. I can see a noticeable lag in release & debug build.

when set this property to unhandled only, Bugsna still write the offending thread(the thread create the error)

my old settings:

Bugsnag.setThreadTracingEnabled(false)
Bugsnag.setWriteBinaryImagesForUserReported(false)
Bugsnag.setSuspendThreadsForUserReported(false)

@twometresteve
Copy link
Contributor

HI @zhaoze1991 - I think it would be best if you could email our support team on [email protected] and they will be very happy to help you. Best regards, Steve.

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.

4 participants