-
Notifications
You must be signed in to change notification settings - Fork 161
Open
Labels
debtCode quality improvement or decrease of technical debt.Code quality improvement or decrease of technical debt.low priorityLow priority issues to be done eventually.Low priority issues to be done eventually.t-core-servicesIssues with this label are in the ownership of the core services team.Issues with this label are in the ownership of the core services team.t-unblockingIssues with this label are in the ownership of the unblocking team.Issues with this label are in the ownership of the unblocking team.
Description
Problem
PR #619 introduced useful HTTP agent pooling functionality, but the current test suite lacks adequate coverage for HTTPS upstreams.
- We currently have limited tooling for comprehensive HTTPS testing, making it difficult to implement full coverage immediately.
- This limitation may be mitigated after the release of feat: implement https server support #602, which is expected to simplify our testing capabilities.
Given the dependency and the good-enough coverage for HTTP, we decided to merge #619 as is, while acknowledging the insufficient coverage for HTTPS.
Proposed Solution
- Conduct a test coverage audit focused specifically on the HTTP/HTTPS agent pooling feature.
- Identify and implement critical missing tests (primarly for HTTPS upstreams).
Refs
Metadata
Metadata
Assignees
Labels
debtCode quality improvement or decrease of technical debt.Code quality improvement or decrease of technical debt.low priorityLow priority issues to be done eventually.Low priority issues to be done eventually.t-core-servicesIssues with this label are in the ownership of the core services team.Issues with this label are in the ownership of the core services team.t-unblockingIssues with this label are in the ownership of the unblocking team.Issues with this label are in the ownership of the unblocking team.