Auth transports add credentials after cross-origin redirects
维护者通常 1 天内回复
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 25/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 停滞
- 技术栈
- go
调研方向
从 BasicAuthTransport 和 UnauthenticatedRateLimitedTransport 开始，尤其关注它们的 RoundTrip 方法，并按照描述使用两个 httptest 服务器复现该行为。完成的标准是：两个 transport 都有回归覆盖，并且重定向到不同 origin 的请求不包含它们添加的授权标头。
由索引模型根据 Issue 内容生成。
描述
Summary
BasicAuthTransport and UnauthenticatedRateLimitedTransport add credentials in RoundTrip for each outgoing request. When Go's net/http client follows a redirect to a different origin, the redirected request can pass through the same transport and receive those credentials again.
Expected behavior
Credentials added by these transports should only be sent to the original origin. Redirected requests that cross origin should continue without these transport-managed credentials.
Reproduction
This can be reproduced with two httptest servers: one server returns a redirect to a second server, and the second server records whether the redirected request contains the transport-added authorization headers.
Proposed fix
PR #4364 avoids adding these credentials on cross-origin redirect requests and includes regression coverage for both affected transports.
- 主要语言
- Go
- 星标
- 11.3k
- 派生
- 2.6k
- 平均合并
- 2 天 8 小时
- 30 天内合并 PR
- 35
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
google/go-github 的其他 Issue
-
Support `reason` in `UsersService.Unsuspend`可能已有人在做 @JamBalaya56562 于 2 天前认领。 未关闭
google/go-github#4613 · 1 个 reaction · 已指派 1 人 ·
维护者通常 1 天内回复
-
Add support for stargazer count and history endpoints可能已有人在做 @ninadsutrave 于 3 天前认领。 未关闭
google/go-github#4610 · 已指派 1 人 ·
维护者通常 1 天内回复
-
http.Client.Timeout is ignored by redirect-aware requests可能已有人在做 @sb123sb123 于 23 天前认领。 未关闭
难度 3/5 1-2 天 新手友好度 68/100
维护者通常 1 天内回复
-
Add Copilot content exclusion set/get enterprise and org set endpoints可能已有人在做 @Tens1des 于 37 天前认领。 未关闭
google/go-github#4526 · 1 条评论 · 1 个 reaction · 已指派 1 人 ·
维护者通常 1 天内回复
-
Add Copilot custom agents enterprise endpoints可能已有人在做 @Tens1des 于 37 天前认领。 未关闭
google/go-github#4524 · 1 个 reaction · 已指派 1 人 ·
维护者通常 1 天内回复
相似的 Issue
-
Say in the changelog that the text-index update cannot be rolled back可能已有人在做 @praetorianer777 今天认领。 未关闭
难度 1/5 1 小时以内 新手友好度 88/100
praetorianer777/GoTome#275 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
SAP/crossplane-provider-btp#1118 ·
维护者通常 1 天内回复
-
benchmark: `make setup` fails because audit-exporter builds with Go 1.25.0 while go.mod requires 1.26.0可能已有人在做 @xsvm 今天认领。 未关闭
难度 1/5 1 小时以内 新手友好度 84/100
volcano-sh/volcano#6097 · 1 条评论 ·
维护者通常 1 天内回复
-
area/infra backlog bug
难度 2/5 1-3 小时 新手友好度 72/100
praetorianer777/Stator#417 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
维护者通常 1 天内回复