Change the http.response.status_code for spans to int

[denrase]

📜 Description

💡 Motivation and Context

Closes #6327

💚 How did you test it?

Unit tests

📝 Checklist

You have to check all boxes before merging:

• I added tests to verify the changes.
• No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
• I updated the docs if needed.
• I updated the wizard if needed.
• Review from the native team if needed.
• No breaking change or entry added to the changelog.
• No breaking change for hybrid SDKs or communicated to hybrid SDKs.

[denrase]

Added to breaking changes, not sure if we should wait after 9.0.0, as this is marked for v10 in linear.

[codecov]

❌ 2 Tests Failed:

Tests completed	Failed	Passed	Skipped
3971	2	3969	52

View the top 2 failed test(s) by shortest run time

SentryTests.SentryFileManagerTests::testCreateDirectoryIfNotExists_successful_shouldNotLogError

Stack Traces | 0s run time

.../SentryTests/Helper/SentryFileManagerTests.swift:1056 - XCTAssertEqual failed: ("1") is not equal to ("0")

SentryTests.SentryHttpTransportFlushIntegrationTests::testFlush_WhenNoEnvelopes_BlocksAndFinishes

Stack Traces | 0s run time

.../SentryTests/Networking/SentryHttpTransportFlushIntegrationTests.swift:25 - XCTAssertLessThan failed: ("0.11904426834000002") is not less than ("0.1")

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}

[github-actions]

Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	1224.34 ms	1256.00 ms	31.66 ms
Size	24.14 KiB	1.01 MiB	1014.69 KiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
7bb24a2	1229.16 ms	1256.66 ms	27.50 ms
2a7868a	1226.54 ms	1256.92 ms	30.37 ms
7b08a87	1220.14 ms	1246.10 ms	25.96 ms
035974f	1225.89 ms	1251.23 ms	25.34 ms
2de3f92	1207.56 ms	1234.96 ms	27.40 ms
705b962	1200.92 ms	1222.04 ms	21.13 ms
42cfd79	1222.13 ms	1244.23 ms	22.10 ms
25f2d2c	1232.02 ms	1242.78 ms	10.76 ms
e8e40fd	1222.10 ms	1259.91 ms	37.80 ms
5c5648e	1234.44 ms	1253.79 ms	19.35 ms

App size

Revision	Plain	With Sentry	Diff
7bb24a2	23.75 KiB	973.68 KiB	949.93 KiB
2a7868a	23.75 KiB	880.20 KiB	856.45 KiB
7b08a87	23.74 KiB	913.70 KiB	889.95 KiB
035974f	23.74 KiB	874.07 KiB	850.33 KiB
2de3f92	23.75 KiB	919.69 KiB	895.94 KiB
705b962	23.74 KiB	995.47 KiB	971.73 KiB
42cfd79	23.75 KiB	880.20 KiB	856.45 KiB
25f2d2c	23.75 KiB	866.69 KiB	842.94 KiB
e8e40fd	23.75 KiB	1022.79 KiB	999.04 KiB
5c5648e	23.75 KiB	879.60 KiB	855.86 KiB

Previous results on branch: fix/int-for-http-status

Startup times

Revision	Plain	With Sentry	Diff
1fefafa	1242.24 ms	1253.32 ms	11.07 ms
0b972e4	1222.66 ms	1259.29 ms	36.63 ms
2b30f77	1217.06 ms	1250.73 ms	33.67 ms
df46e67	1218.75 ms	1246.72 ms	27.97 ms
2e40605	1227.63 ms	1256.78 ms	29.15 ms

App size

Revision	Plain	With Sentry	Diff
1fefafa	24.14 KiB	1.01 MiB	1013.53 KiB
0b972e4	24.14 KiB	1.01 MiB	1015.51 KiB
2b30f77	24.14 KiB	1.01 MiB	1013.53 KiB
df46e67	24.14 KiB	1.01 MiB	1012.68 KiB
2e40605	24.14 KiB	1.01 MiB	1012.85 KiB

[philprime]

@denrase this is a breaking change, we didn't think we have the time to do it for the v9 release so it was pushed back for v10 which won't happen for a while.

As you already did it now, it makes sense to consider it for the next v9 RC

[philprime]

LGTM

[philipphofmann]

It's a small breaking change. I think it's OK, as it's almost a bugfix actually. Thanks

[github-actions]

🚨 Detected changes in high risk code 🚨

High-risk code can easily blow up and is hard to test. We had severe bugs in the past. Be extra careful when changing these files, and have an extra careful look at these:

• Sources/Sentry/SentryNetworkTracker.m