Skip to content

feat(boto3): Add S3 extension - #7888

Merged
pabloDeputter merged 31 commits into
major/3.0from
pablo/boto3-add-s3-extension
Oct 8, 2026
Merged

pabloDeputter merged 31 commits into
major/3.0from
pablo/boto3-add-s3-extension

Conversation

@pabloDeputter

@pabloDeputter pabloDeputter commented Oct 6, 2026 •

Copy link
Copy Markdown
Member

Description

Adds the following request/response attributes: aws.s3.bucket, aws.s3.key, aws.s3.upload_id, aws.s3.copy_source, aws.s3.delete, aws.s3.part_number, http.response.body.size (number of bytes in payload), file.size (represents the total file/object size)

Following tests were removed/moved from old test_s3.py (old boto3 integration only added s3 streaming responses support):

  • test_basic() removed - basically only tests botocore xml parsing behavior; generic span behavior is already tested in test_client.py.
  • test_streaming() / test_streaming_close() removed - behavior is already tested in test_client.py with mock HTTP server.
  • test_span_origin() removed - overlaps with existing tests.
  • test_omit_url_data_if_parsing_fails() moved to test_client.py and simplified since it's generic instrumentation.
  • test_breadcrumb() and test_url_query_data_collection_breadcrumb() moved and combined to test_client.py, once again since it's generic instrumentation.

Issues

Resolves #7576

@github-actions

github-actions Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Codecov Results 📊

✅ 55462 passed | ⏭️ 2755 skipped | Total: 58217 | Pass Rate: 95.27% | Execution Time: 152m 43s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -82476
Passed Tests 📉 -77955
Failed Tests —
Skipped Tests 📉 -4521

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2061 uncovered lines.
❌ Project coverage is 90.06%. Comparing base (0cc9b31) to head (17469ef).

Coverage diff
@@            Coverage Diff             @@
##        master       #PR       +/-##
==========================================
- Coverage    90.37%    90.06%    -0.31%
==========================================
  Files          202       187       -15
  Lines        26552     20733     -5819
  Branches      9854      7136     -2718
==========================================
+ Hits         23995     18672     -5323
- Misses        2557      2061      -496
- Partials      1495      1214      -281

Generated by Codecov Action

@pabloDeputter
pabloDeputter force-pushed the pablo/boto3-add-s3-extension branch from 08ce57f to c43fff1 Compare October 6, 2026 14:53
@pabloDeputter
pabloDeputter force-pushed the pablo/boto3-add-s3-extension branch from ca494d2 to 9938a64 Compare October 7, 2026 12:47
@pabloDeputter
pabloDeputter marked this pull request as ready for review October 7, 2026 13:38
@pabloDeputter
pabloDeputter requested a review from a team as a code owner October 7, 2026 13:38

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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 9eb12cd. Configure here.

Comment thread sentry_sdk/integrations/boto3/_services/s3.py Outdated

@ericapisani ericapisani left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, just saw this one small (non-blocking) 🧹

Comment thread tests/integrations/boto3/test_client.py Outdated
@pabloDeputter
pabloDeputter merged commit 3168476 into major/3.0 Oct 8, 2026
147 checks passed
@pabloDeputter
pabloDeputter deleted the pablo/boto3-add-s3-extension branch October 8, 2026 09:31
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.

2 participants