Add HTTPS Test Coverage for HTTP Agent Pooling

未关闭
#620 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
40/100
Issue 类型
功能
描述清晰度
基本清楚
活跃度
停滞

调研方向

The payload names no test files. Start with the agent pooling changes from PR #619 and the existing HTTP upstream tests, then run the test suite with coverage to see which HTTPS upstream paths are uncovered. Check whether the test tooling from proxy-chain PR #602 has landed, since the issue says HTTPS testing depends on it. Done means the audit is written up and the critical HTTPS upstream cases have passing tests.

由索引模型根据 Issue 内容生成。

描述

debt low priority t-unblocking

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 https://github.com/apify/proxy-chain/pull/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

主要语言
TypeScript
星标
1k
派生
167
平均合并
18 小时 52 分钟
30 天内合并 PR
9

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

apify/proxy-chain 的其他 Issue

查看 apify/proxy-chain 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。