implement request factory

[wenxi-zeng]

No description provided.

[codecov-commenter]

Codecov Report

Patch coverage: 88.13% and project coverage change: -0.11 ⚠️

Comparison is base (86a4efa) 79.09% compared to head (7e743da) 78.99%.

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #176      +/-   ##
============================================
- Coverage     79.09%   78.99%   -0.11%     
- Complexity      481      483       +2     
============================================
  Files            76       76              
  Lines          6334     6379      +45     
  Branches        785      785              
============================================
+ Hits           5010     5039      +29     
- Misses          689      706      +17     
+ Partials        635      634       -1     

Impacted Files	Coverage Δ
...m/segment/analytics/kotlin/core/HTTPClientTests.kt	86.36% <78.26%> (-4.34%)	⬇️
...va/com/segment/analytics/kotlin/core/HTTPClient.kt	72.61% <93.33%> (-10.46%)	⬇️
...com/segment/analytics/kotlin/core/Configuration.kt	94.44% <100.00%> (+0.32%)	⬆️
...java/com/segment/analytics/kotlin/core/Settings.kt	75.43% <100.00%> (ø)
...alytics/kotlin/core/compat/ConfigurationBuilder.kt	100.00% <100.00%> (ø)
...nt/analytics/kotlin/core/platform/EventPipeline.kt	96.26% <100.00%> (+0.93%)	⬆️
...ics/kotlin/core/compat/ConfigurationBuilderTest.kt	98.59% <100.00%> (+0.02%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[didiergarcia]

Should we make these configurable? Maybe in the constructor with these values as defaults?

[wenxi-zeng]

yes, it is configurable. user can always override openConnection to configure.