Skip to content

Introduce HTTP interface#43

Merged
dorin-ga merged 60 commits intomainfrom
http_interface
May 4, 2026
Merged

Introduce HTTP interface#43
dorin-ga merged 60 commits intomainfrom
http_interface

Conversation

@dorin-ga
Copy link
Copy Markdown
Collaborator

@dorin-ga dorin-ga commented Apr 1, 2026

Separate HTTP implementation in order to more easily manage dependencies and support other libraries/platforms (e.g: consoles). The default HTTP implementation remains CURL.

  • create abstract HTTP interface
  • have HTTP API be agnostic of http implementation
  • create the default CURL interface
  • allow registering a custom interface
  • better dependency management
  • write tests

@dorin-ga dorin-ga requested a review from Gogoshika-ga April 1, 2026 11:48
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 1, 2026

LCOV of commit 681c7e8 during Test Coverage #135

Summary coverage rate:
  lines......: 40.5% (1149 of 2835 lines)
  functions..: 52.6% (211 of 401 functions)
  branches...: no data found

Files changed coverage rate: n/a

Comment thread source/gameanalytics/Http/GAHttpCurl.cpp Outdated
Comment thread source/gameanalytics/GAHTTPApi.cpp Outdated
@dorin-ga dorin-ga requested a review from Gogoshika-ga April 30, 2026 09:23
@dorin-ga dorin-ga requested a review from Gogoshika-ga May 4, 2026 10:28
@dorin-ga dorin-ga changed the title [WIP] Introduce HTTP interface Introduce HTTP interface May 4, 2026
@dorin-ga dorin-ga merged commit 06657bd into main May 4, 2026
25 checks passed
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