Skip to content

chore: upgrade python package and log exceptions #31062

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

Merged
merged 13 commits into from
Apr 11, 2025

Conversation

daibhin
Copy link
Contributor

@daibhin daibhin commented Apr 10, 2025

Problem

We want access to exceptions if the web app isn't available. Let's log them before sending if enabled

Changes

  • Upgrade posthog python package
  • Enable log_captured_exceptions

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR enables exception logging in PostHog by upgrading the posthoganalytics package and configuring it to log captured exceptions.

  • Added posthoganalytics.log_captured_exceptions = True in /posthog/apps.py to enable exception logging
  • Updated posthoganalytics dependency from 3.21.0 to >=3.24.0 in pyproject.toml
  • Works with existing exception autocapture functionality (already enabled with posthoganalytics.enable_exception_autocapture = True)
  • Allows the team to access exception logs even when the web application is unavailable

2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@daibhin daibhin enabled auto-merge (squash) April 11, 2025 16:07
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (wasn't pushed!)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

Copy link
Contributor

github-actions bot commented Apr 11, 2025

Size Change: -1.44 kB (-0.01%)

Total Size: 13.2 MB

Filename Size Change
frontend/dist/toolbar.js 13.2 MB -1.44 kB (-0.01%)

compressed-size-action

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 7)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 7)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@daibhin daibhin merged commit dfa6489 into master Apr 11, 2025
108 of 109 checks passed
@daibhin daibhin deleted the dn-chore/upgrade-python-package branch April 11, 2025 18:45
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