Repository navigation
fix(http): 恢复 #173 请求整体超时修复供重新审阅 - #181
Closed
3316891527 wants to merge 1 commit into
Closed
3316891527 wants to merge 1 commit into
3316891527 wants to merge 1 commit into
Conversation
2 of 5 tasks
3316891527
added
the
ci:full-android
3316891527
force-pushed
the
fix/reopen-pr-173-http-timeout
branch
from
November 9, 2026 03:16
f985d7a to
a1db67c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
变更说明 / Description
为 HTTP 请求设置 OkHttp callTimeout，使整体超时覆盖请求的完整生命周期；同步 TypeScript、示例 JavaScript 与内置脚本的 timeout_ms 参数透传。
背景与动机 / Context and motivation
维护者要求撤销 #170–#176 并恢复 open 审阅。原 #173 已合并且无法重开，本 PR 恢复其完整补丁。未经维护者明确允许不得合并或启用自动合并。
改动范围 / Changes
恢复 StandardHttpTools、系统工具说明以及两份 JavaScript 和 TypeScript 接口，共 5 文件。保留约定：timeout >= 1000 按毫秒，较小值按秒；timeout_ms 固定为毫秒；向上取整并封顶 600 秒。
兼容性与风险 / Compatibility and risks
保留旧 timeout 参数语义，新增明确的毫秒参数；整体超时可能终止此前能持续更久的调用。当前暂以撤销 PR #177 的分支为目标，仅展示本项差异；待 #177 获准进入 dev 后应改为 dev 并重新检查，不能合入临时基准分支。
关联 Issue / Related issue
原 #173、上游问题 AAswordman#1202；撤销要求见 #177。
验证方式 / Verification
初次重建时已逐项核对原完整合并补丁。本轮在清理历史空白并补齐 CI 的共同回滚基准上重放该项，保持独立功能差异。当前 PR 启用 ci:full-android，要求固定合并候选实际运行 assembleDebug 与 :app:testDebugUnitTest，Candidate checks 必须检查两项真实成功。新的结果以本 PR Checks 为准；历史 CI 不代替本轮构建和测试。
证据 / Evidence
原实现与历史检查见 #173，回滚基准见 #177。
检查清单 / Checklist
本项已随 集成 PR #191 合入
dev，并经 发布同步 PR #193 同步到main。原来源提交a1db67cb030c9282208bd3310a5fcdcd627c99da已核对保留。本 PR 原目标为
fix/revert-pr-170-176；由于dev已包含全部来源提交，GitHub 拒绝将目标改到dev。关闭此重复待合并条目，代码与原始审阅记录均保留。