feat(boto3): Support span streaming

[alexander-alderman-webb]

Description

In the streaming path, use

• http.request.method instead of http.method
• url.full instead of `url
• url.query instead of aws.request.url
• url.fragment instead of http.fragment
• rpc.method instead of both aws.service_id and aws.operation_name

---

Adapting Tests

sed commands used for converting transaction context managers:

• sed -i '' 's/sentry_sdk.start_transaction()/sentry_sdk.traces.start_span(name="custom parent")/g'
• sed -i '' 's/sentry_sdk.start_transaction() as transaction/sentry_sdk.traces.start_span(name="custom parent") as span/g'
• sed -i '' 's/start_transaction() as transaction/sentry_sdk.traces.start_span(name="custom parent") as span/g'

sed commands used for converting specific attributes:

• sed -i '' 's/http.method/http.request.method/g'
• sed -i '' 's/aws.request.url/url.full/g'
• sed -i '' 's/http.fragment/url.fragment/g'
• sed -i '' 's/http.query/url.query/g'

sed commands used for converting event capture:

• sed -i '' 's/assert len(spans) == 2/assert len(spans) == 3/g'
• sed -i '' 's/assert len(spans) == 1/assert len(spans) == 2/g'
• sed -i '' 's/(event, ) = (item.payload for item in items if item.type == "span")/spans = [item.payload for item in items if item.type == "span"]/g'
• sed -i '' 's/event["spans"]/spans/g'
• sed -i '' '/assert event["type"] == "transaction"/d'
• sed -i '' 's/transaction.finish()/span.finish()/g'
• sed -i '' 's/(event,) = events/(event, ) = (item.payload for item in items if item.type == "span")/g'
• sed -i '' 's/events = capture_events()/items = capture_items("event", "transaction", "span")/g'
• sed -i '' 's/capture_events,/capture_items,/g'

sed commands used for converting op:

• sed -i '' 's/["op"]/["attributes"]["sentry.op"]/g'

sed commands used for converting origin:

• sed -i '' 's/["origin"]/["attributes"]["sentry.origin"]/g'

sed commands used for converting description:

• sed -i '' 's/description/name/g'

sed commands used for converting data to attributes:

• sed -i '' 's/["data"]/["attributes"]/g'

Issues

Reminders

• Please add tests to validate your changes, and lint your code using tox -e linters.
• Add GH Issue ID & Linear ID (if applicable)
• PR title should use conventional commit style (feat:, fix:, ref:, meta:)
• For external contributors: CONTRIBUTING.md, Sentry SDK development docs, Discord community

[github-actions]

Codecov Results 📊

✅ 146 passed | Total: 146 | Pass Rate: 100% | Execution Time: 21.64s

All tests are passing successfully.

❌ Patch coverage is 9.09%. Project has 14148 uncovered lines.

Files with missing lines (2)

File	Patch %	Lines
boto3.py	13.13%	⚠️ 86 Missing
consts.py	99.48%	⚠️ 2 Missing

---

Generated by Codecov Action

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

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

^{Reviewed by Cursor Bugbot for commit 7e39d58. Configure here.}

[ericapisani]

For variable names capturing states (like this one), naming it like a noun/a thing makes it harder to read in isolation in other areas of the code.

Something that leverages a verb/predicate, along the lines of is_span_streaming_enabled, is generally better for values/variables like this.

[ericapisani]

Because this is set outside of any conditionals within this block, you could move this up into the attributes dict passed in start_span above

[ericapisani]

I noticed that the legacy approach doesn't have this guard, and I assume it's because set_data is typed more permissively in that it accepts None where set_attribute does not.

This makes me wonder - are we currently sending None values within the envelope for legacy spans for the HTTP_METHOD on line 106? I wonder if we should also guard that in order to remove an unnecessary value being added 🤔

[ericapisani]

As this is currently written, NoOpStreamedSpan spans will evaluate to true and enter this block. It doesn't look like it'll cause the application to crash as it does have a name property (although it's an empty string), but wanted to call this out in case this is unintentional.

If you only care about StreamedSpans entering this block, then this will need to be updated to type(span) is StreamedSpan.