Skip to content

feat: Add aiohttp test project#7

Merged
ericapisani merged 2 commits into
mainfrom
aiohttp-project-01qf4
Apr 29, 2026
Merged

feat: Add aiohttp test project#7
ericapisani merged 2 commits into
mainfrom
aiohttp-project-01qf4

Conversation

@ericapisani
Copy link
Copy Markdown
Member

Add a minimal aiohttp app wired up to the local sentry-python SDK with the AioHttpIntegration enabled.

The app exposes three routes used to exercise the integration end-to-end against a local SDK build:

  • / returns a JSON index pointing at the other routes
  • /error raises an unhandled ValueError after setting a user, to verify error capture and user scoping
  • /http-error raises web.HTTPInternalServerError to verify how aiohttp HTTP-level errors flow through

A no-op middleware is registered to confirm middleware ordering with the integration. pyproject.toml pins sentry-sdk to the sibling ../../sentry-python checkout via [tool.uv.sources] so the sample runs against in-progress SDK changes, and run.sh bootstraps uv if missing and starts the server on port 5000.

.gitignore is also extended to skip .claude/settings.local.json and .serena so local agent tooling files don't leak into commits.

Add a minimal aiohttp app wired up to the local sentry-python SDK with the
AioHttpIntegration enabled. Includes routes for a successful response, an
unhandled exception, and an HTTPInternalServerError to exercise error and
trace capture.

Also extend .gitignore to skip local Claude and Serena tooling files.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ericapisani ericapisani marked this pull request as ready for review April 29, 2026 11:47
Comment thread test-aiohttp/main.py Outdated
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 44760e9. Configure here.

Comment thread test-aiohttp/main.py
@ericapisani ericapisani merged commit f9b7f31 into main Apr 29, 2026
8 checks passed
@ericapisani ericapisani deleted the aiohttp-project-01qf4 branch April 29, 2026 11:59
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.

1 participant