Consider wpcom_vip_get_resized_remote_image_url() as auto-escaping if fourth argument is truthy
还没有人认领这个 Issue。
评估
调研方向
先从 VIP Go 规则集的 XSS 输出转义处理以及 issue 中描述的 $customAutoEscapedFunctions 配置入手，然后检查链接的 vip-deprecated.php 函数行为。为第四个参数添加条件处理，并验证所提供的 true/默认值和 false 示例是否产生预期的警告。
由索引模型根据 Issue 内容生成。
描述
Bug Description
The VIP Go standard uses WordPress.Security.EscapeOutput.OutputNotEscaped but doesn't recognize that the deprecated wpcom_vip_get_resized_remote_image_url() function auto-escapes if it's fourth argument is true (default).
Since it's conditional, we can't just add an entry for it to $customAutoEscapedFunctions in the VIP Go ruleset (see here) as otherwise it wouldn't get flagged if the fourth argument was not truthy.
Minimal Code Snippet
// Should not be flagged.
<img src="<?php echo wpcom_vip_get_resized_remote_image_url($item->image_url, 360,270); ?>" alt="<?php echo esc_attr($item->name) ?>">
// Should be flagged.
<img src="<?php echo wpcom_vip_get_resized_remote_image_url($item->image_url, 360,270, false); ?>" alt="<?php echo esc_attr($item->name) ?>">
Tested Against master branch?
- I have verified the issue still exists in the
masterbranch of VIPCS. - I have verified the issue still exists in the
developbranch of VIPCS.
- 主要语言
- PHP
- 星标
- 261
- 派生
- 44
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
Automattic/VIP-Coding-Standards 的其他 Issue
-
AlwaysReturnInFilter: isInsideIfConditonal() guards the conditions array after reading it可能已有人在做 @tomjn 于 6 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 78/100
-
Bug: PreGetPosts warns when the early is_main_query() return is not the first statement in its if可能已有人在做 @tomjn 于 6 天前认领。 未关闭
难度 3/5 1-2 天 新手友好度 76/100
-
Suppress filters in get_posts false positive可能已有人在做 @tomjn 于 8 天前认领。 未关闭
难度 3/5 1-2 天 新手友好度 68/100
-
难度 3/5 1-2 天 新手友好度 48/100
-
Breaking Change Type: Maintenance
难度 4/5 3-5 天 新手友好度 35/100
Automattic/VIP-Coding-Standards#849 · 1 条评论 ·
查看 Automattic/VIP-Coding-Standards 的全部 Issue
相似的 Issue
-
maintenance
难度 2/5 1-3 小时 新手友好度 62/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
hawk-digital-environments/HAWKI#443 ·
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 78/100
crazy-goat/rabbit-stream#799 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 72/100
-
Code Quality
难度 2/5 1-3 小时 新手友好度 76/100
Automattic/safe-publish#708 ·
维护者通常 1 天内回复