Skip to content

Commit

Permalink
fix: remove sentry transaction sampling (#115)
Browse files Browse the repository at this point in the history
(cherry picked from commit 47ca357)
  • Loading branch information
JustARatherRidiculouslyLongUsername committed Nov 12, 2024
1 parent d73140b commit f00ce32
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions quickbooks_desktop_api/sentry.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ def init():
send_default_pii=True,
integrations=[DjangoIntegration()],
environment=os.environ.get('SENTRY_ENV'),
traces_sampler=Sentry.traces_sampler,
attach_stacktrace=True,
before_send=Sentry.before_send,
request_bodies='small',
Expand All @@ -27,15 +26,6 @@ def init():
'fyle_rest_auth'],
)

@staticmethod
def traces_sampler(sampling_context):
# avoiding ready APIs in performance tracing
if sampling_context.get('wsgi_environ') is not None:
if 'ready/' in sampling_context['wsgi_environ']['PATH_INFO']:
return 0

return 1

@staticmethod
def before_send(event, hint):
if 'exc_info' in hint:
Expand Down

0 comments on commit f00ce32

Please sign in to comment.