Skip to content

Comments

[#41] Implement simple networking stack based on libcurl#42

Merged
RippeR37 merged 3 commits intodevelopfrom
ftr/net/1
Mar 30, 2025
Merged

[#41] Implement simple networking stack based on libcurl#42
RippeR37 merged 3 commits intodevelopfrom
ftr/net/1

Conversation

@RippeR37
Copy link
Owner

This commit adds simple networking functionality built on top of libcurl library. Main implementation focuses on introduction of new (optional) thread (NetThread) that will be used to process networking I/O operations and some simple utilities (SimpleUrlLoader class) that can be used to make network requests.

For now only simple functionality exists:

  • Simple fetching via GET/HEAD (single callback when request finishes)
  • Optionally providing custom headers for the requests
  • Reading response headers
  • Specifying timeout details
  • Reading response timing details

This will be expanded in the followup commits.

@RippeR37 RippeR37 self-assigned this Mar 30, 2025
@RippeR37 RippeR37 added the enhancement New feature or request label Mar 30, 2025
@RippeR37 RippeR37 linked an issue Mar 30, 2025 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Mar 30, 2025

Codecov Report

Attention: Patch coverage is 2.57732% with 189 lines in your changes missing coverage. Please review.

Project coverage is 60.01%. Comparing base (5fa2e8c) to head (277f490).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
src/base/net/impl/net_thread_impl.cc 0.00% 158 Missing ⚠️
src/base/net/impl/net_thread.cc 0.00% 18 Missing ⚠️
src/base/init.cc 41.66% 7 Missing ⚠️
src/base/net/simple_url_loader.cc 0.00% 6 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop      #42       +/-   ##
============================================
- Coverage    70.22%   60.01%   -10.21%     
============================================
  Files           49       52        +3     
  Lines         1088     1278      +190     
  Branches        55       81       +26     
============================================
+ Hits           764      767        +3     
- Misses         324      511      +187     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This commit adds simple networking functionality built on top of libcurl
library. Main implementation focuses on introduction of new (optional)
thread (NetThread) that will be used to process networking I/O
operations and some simple utilities (`SimpleUrlLoader` class) that can
be used to make network requests.

For now only simple functionality exists:

* Simple fetching via GET/HEAD (single callback when request finishes)
* Optionally providing custom headers for the requests
* Reading response headers
* Specifying timeout details
* Reading response timing details

This will be expanded in the followup commits.
@RippeR37 RippeR37 merged commit bd8063b into develop Mar 30, 2025
37 of 39 checks passed
@RippeR37 RippeR37 deleted the ftr/net/1 branch March 30, 2025 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement networking module

1 participant