URI::MailTo::EMAIL_REGEXP matches whole strings but URI::RFC2396_PARSER.make_regexp matches partial strings
还没有人认领这个 Issue。
评估
调研方向
审查公共入口点 URI::RFC2396_PARSER.make_regexp 及其与 URI::MailTo::EMAIL_REGEXP 的交互;首先确定现有的匹配语义和可用测试。Done 应在保留当前默认行为的同时,提供一个显式的完全匹配选项,其覆盖范围应拒绝尾随空白字符。
由索引模型根据 Issue 内容生成。
描述
The way to test if a string is a valid email address is to directly use URI::MailTo::EMAIL_REGEXP
require 'uri'
puts URI::MailTo::EMAIL_REGEXP.match?('me@example.com') # true
puts URI::MailTo::EMAIL_REGEXP.match?('me@example.com ') # false (trailing space)
However, the correct way to validate that a string is a valid URL is to wrap the regex in anchors:
require 'uri'
puts /\A#{URI::RFC2396_PARSER.make_regexp}\z/.match?('https://example.com/') # true
puts /\A#{URI::RFC2396_PARSER.make_regexp}\z/.match?('https://example.com/ ') # false (trailing space)
If I directly use make_regexp, that second string matches because of a partial match.
require 'uri'
puts URI::RFC2396_PARSER.make_regexp.match?('https://example.com/ ') # true (trailing space)
Same behavior if I pass schemes into make_regexp or use the too. I'm on Ruby 4.0.1 but I don't think this behavior changed recently.
Would it be okay if I made a PR adding a keyword argument to make_regexp? It would get called like:
URI::RFC2396_PARSER.make_regexp(full_match: true).match?('https://example.com/ ') # true (trailing space)
I figured changing the default behavior could be a breaking change for a lot of people and that doesn't feel worth it.
- 主要语言
- Ruby
- 星标
- 125
- 派生
- 65
- 平均合并
- 6 小时 4 分钟
- 30 天内合并 PR
- 2
贡献指南
这个仓库没有索引到贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
ruby/uri 的其他 Issue
-
难度 3/5 1-2 天 新手友好度 48/100
-
难度 1/5 1 小时以内 新手友好度 25/100
-
难度 3/5 1-2 天 新手友好度 68/100
-
难度 3/5 1-2 天 新手友好度 52/100
-
难度 5/5 一周以上 新手友好度 35/100
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 70/100
-
bug
难度 1/5 1 小时以内 新手友好度 90/100
riscv/riscv-unified-db#2626 ·
-
Component: GLib
难度 2/5 1-3 小时 新手友好度 70/100
-
ds-drift
难度 2/5 1-3 小时 新手友好度 70/100
we-promise/sure#3693 ·
-
难度 2/5 1-3 小时 新手友好度 75/100
simp/pupmod-simp-simp#395 ·