Skip to content

Http module#831

Merged
MartinCardozo-SDK merged 35 commits intodevelopmentfrom
http-module
Feb 13, 2026
Merged

Http module#831
MartinCardozo-SDK merged 35 commits intodevelopmentfrom
http-module

Conversation

@MartinCardozo-SDK
Copy link
Contributor

@MartinCardozo-SDK MartinCardozo-SDK commented Feb 12, 2026

iOS SDK

What did you accomplish?

How do we test the changes introduced in this PR?

Extra Notes

gthea and others added 23 commits January 24, 2026 21:03
@gthea
Logging module
ae01cc3
@gthea
Move Logger to Logging (git mv)
4898e7a
@gthea
Logging module extracted
3929de1
@gthea
Update README
e962e3e
@gthea
Remove unnecessary imports
c14ee52
@gthea
Restore Split Logger shim
200833a
@gthea
Fix comments
7dd76dd
@gthea
Remove unnecessary rename
5a412df
@gthea
Expose Logging as product
83f8ded
@gthea
SplitModules product
1293b49
@gthea
WIP kaban
57f0791
@gthea
Umbrella module
74bd67c
@gthea
Merge branch 'logging-module' of github.com:splitio/ios-client into l…
f4cae2d 
…ogging-module
@gthea
Update README
e41ac65
@gthea
Update Split podspec
1c3e531
@gthea
Merge with master
87833da
@gthea
Improve comments
f3d5f84
@gthea
Add Logging dependency to watchOs
75dc3a0
@gthea
Increase test coverage
342afb5
@gthea
Add Logging tests to sonar report
9b7b7eb
@gthea
http scaffold
3b21b27
@MartinCardozo-SDK @cursoragent
Extract Http module from Split/Network to Sources/Http
a32dad7 
Move HTTP-related code into a standalone Http module for better
modularity. This includes HttpClient, request/response types,
TLS pinning infrastructure, and endpoint handling. The Split
module now imports Http and uses adapter protocols for
authentication and notifications.

Co-authored-by: Cursor <cursoragent@cursor.com>
@MartinCardozo-SDK
Http module extracted
62f6124
@MartinCardozo-SDK MartinCardozo-SDK self-assigned this Feb 12, 2026
@MartinCardozo-SDK MartinCardozo-SDK requested a review from a team as a code owner February 12, 2026 13:38
@devops-split
Copy link

devops-split commented Feb 12, 2026

SDK Readiness Test Results ✅

Metric http-module P50 development P50 Significance
SDK_READY 104 ms 104 ms -
SDK_READY_FROM_CACHE 89 ms 87 ms -

Legend: 🟢 significant improvement | 🔴 significant regression | ⚪ inconclusive (p≥0.05)

ios-test-mock

  • Mock Server: true

  • Iterations: 40

@devops-split
Copy link

devops-split commented Feb 12, 2026

SDK Readiness Test Results ✅

Metric http-module P50 development P50 Significance
SDK_READY 111 ms 103 ms -
SDK_READY_FROM_CACHE 89 ms 85 ms -

Legend: 🟢 significant improvement | 🔴 significant regression | ⚪ inconclusive (p≥0.05)

ios-test-mock

  • Mock Server: true

  • Iterations: 40

@devops-split
Copy link

devops-split commented Feb 12, 2026

SDK Readiness Test Results ✅

Metric http-module P50 development P50 Significance
SDK_READY 106 ms 99 ms -
SDK_READY_FROM_CACHE 90 ms 87 ms -

Legend: 🟢 significant improvement | 🔴 significant regression | ⚪ inconclusive (p≥0.05)

ios-test-mock

  • Mock Server: true

  • Iterations: 40

@devops-split
Copy link

devops-split commented Feb 12, 2026

SDK Readiness Test Results ✅

Metric http-module P50 development P50 Significance
SDK_READY 103 ms 100 ms -
SDK_READY_FROM_CACHE 88 ms 88 ms -

Legend: 🟢 significant improvement | 🔴 significant regression | ⚪ inconclusive (p≥0.05)

ios-test-mock

  • Mock Server: true

  • Iterations: 40

gthea
gthea approved these changes Feb 12, 2026
View reviewed changes
@devops-split
Copy link

devops-split commented Feb 13, 2026

SDK Readiness Test Results ✅

Metric http-module P50 development P50 Significance
SDK_READY 107 ms 103 ms -
SDK_READY_FROM_CACHE 85 ms 86 ms -

Legend: 🟢 significant improvement | 🔴 significant regression | ⚪ inconclusive (p≥0.05)

ios-test-mock

  • Mock Server: true

  • Iterations: 40

MartinCardozo-SDK and others added 2 commits February 13, 2026 13:32
@MartinCardozo-SDK @cursoragent
Add unit tests for HttpError and HttpDataRequest
90fb560 
Tests cover:
- HttpError: isProxyOutdatedError logic, internalCode handling, code extraction
- HttpDataRequest: data chunk accumulation, completion/error flows, pinning credential fail handling

Co-authored-by: Cursor <cursoragent@cursor.com>
@MartinCardozo-SDK @cursoragent
Add Http module tests and README to Xcode project
80ca322 
Adds file references for all existing test files, mocks, and README.md
so they are visible in Xcode's project navigator.

Co-authored-by: Cursor <cursoragent@cursor.com>
@devops-split
Copy link

devops-split commented Feb 13, 2026

SDK Readiness Test Results ✅

Metric http-module P50 development P50 Significance
SDK_READY 105 ms 102 ms -
SDK_READY_FROM_CACHE 86 ms 84 ms -

Legend: 🟢 significant improvement | 🔴 significant regression | ⚪ inconclusive (p≥0.05)

ios-test-mock

  • Mock Server: true

  • Iterations: 40

@devops-split
Copy link

devops-split commented Feb 13, 2026

SDK Readiness Test Results ✅

Metric http-module P50 development P50 Significance
SDK_READY 101 ms 101 ms -
SDK_READY_FROM_CACHE 86 ms 88 ms -

Legend: 🟢 significant improvement | 🔴 significant regression | ⚪ inconclusive (p≥0.05)

ios-test-mock

  • Mock Server: true

  • Iterations: 40

@sonarqube-pull-requests
Copy link

sonarqube-pull-requests bot commented Feb 13, 2026

Quality Gate failed Quality Gate failed

Failed conditions
2 New issues
78.5% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE SonarQube for IDE

@MartinCardozo-SDK MartinCardozo-SDK merged commit 7aec5ed into development Feb 13, 2026
64 of 67 checks passed
@MartinCardozo-SDK MartinCardozo-SDK deleted the http-module branch February 13, 2026 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gthea gthea gthea approved these changes

Assignees

@MartinCardozo-SDK MartinCardozo-SDK

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@MartinCardozo-SDK @devops-split @gthea