`URI::FTP#normalize` returns incorrect (?) value if original path is empty

未关闭

#185 创建于 2025年10月14日 4 条评论 0 个 reaction 已指派 0 人在 GitHub 查看

还没有人认领这个 Issue。

评估

难度: 3/5
预计耗时: 1-2 天
新手友好度: 45/100

Issue 类型: 缺陷
描述清晰度: 基本清楚
活跃度: 停滞
技术栈: ruby
领域: tooling

调研方向

从 lib/uri/generic.rb 中的 URI::Generic#normalize! 和 lib/uri/ftp.rb 中的 URI::FTP#set_path 开始，将链接的代码行作为切入点。检查 RFC 1738，并为路径为空的 FTP URI 添加针对性的规范化测试。完成的标准是行为已经确定，并且默认目录不再被错误编码为 %2F。

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

描述

I don't know how common this case is, but I did run into it accidentally while testing some code.

u = URI("ftp://ftp.example.com") # Note that there's no slash at the end
u.normalize.to_s # => "ftp://ftp.example.com/%2F"

I would expect the result to be "ftp://ftp.example.com/" or possibly the same as the original string, "ftp://ftp.example.com". As best as I can tell from RFC 1738, Section 3.2, "ftp://ftp.example.com" and "ftp://ftp.example.com/" are equivalent and refer to whatever the "default" directory is for FTP access, while "ftp://ftp.example.com/%2F" refers to the root directory.

This seems to be caused by a combination of two things. First, this code in URI::Generic#normalize!calls set_path('/') if the path is empty:
https://github.com/ruby/uri/blob/52077e9b07c555de6ad7ee74663b988fa38ca545/lib/uri/generic.rb#L1341-L1343

Second, URI::FTP overrides #set_path so that a leading slash in the argument is percent-encoded and appended to an unencoded slash: https://github.com/ruby/uri/blob/52077e9b07c555de6ad7ee74663b988fa38ca545/lib/uri/ftp.rb#L245-L247

I didn't see any tests for normalizing FTP URIs, so I'm not sure if this behavior is intended.

主要语言: Ruby
星标: 125
派生: 65
平均合并: 6 小时 4 分钟
30 天内合并 PR: 2

贡献指南

这个仓库没有索引到贡献指南

从这里开始

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

ruby/uri 的其他 Issue

missing test suite in gem on rubygems 未关闭

难度 3/5 1-2 天新手友好度 48/100

ruby/uri#244 · 2026年8月7日创建
upstream contact for Debian 未关闭

难度 1/5 1 小时以内新手友好度 25/100

ruby/uri#243 · 1 条评论 · 2026年8月7日创建
`URI::MailTo.build` does not percent-encode addresses (contrary to RFC 6068) 未关闭

难度 3/5 1-2 天新手友好度 68/100

ruby/uri#238 · 2026年7月4日创建
Use URI::RFC2396_PARSER.escape explicitly. 未关闭

难度 3/5 1-2 天新手友好度 52/100

ruby/uri#235 · 2026年6月10日创建
Question: how to deal with Regexp::Timeout in _decode_uri_component? 未关闭

难度 5/5 一周以上新手友好度 35/100

ruby/uri#233 · 2026年6月4日创建

查看 ruby/uri 的全部 Issue

相似的 Issue

Missing hu metadata: docs (building:tooling/analyzers/feedback-guidelines:title) 未关闭

难度 2/5 1-3 小时新手友好度 70/100

exercism/website#9643 · 2026年9月24日创建
[Bug] `NUM_EXTERNAL_GUEST_INTERRUPTS` (`GEILEN`) minimum should be 0 未关闭

bug

难度 1/5 1 小时以内新手友好度 90/100

riscv/riscv-unified-db#2626 · 2026年9月24日创建
[GLib] Rename run-end encoded array values accessor to physical_values 未关闭

Component: GLib

难度 2/5 1-3 小时新手友好度 70/100

apache/arrow#51479 · 2026年9月24日创建
DS Drift Patrol — Week of 2026-09-21 未关闭

ds-drift

难度 2/5 1-3 小时新手友好度 70/100

we-promise/sure#3693 · 2026年9月24日创建
Replace Facter APIs deprecated in OpenFact 6 (Facter::Core::Execution.exec) 未关闭

难度 2/5 1-3 小时新手友好度 75/100

simp/pupmod-simp-simp#395 · 2026年9月23日创建

更多 Ruby Issue