security: user-controlled URL params injected directly into Redux store via JSON.parse
维护者通常 1 天内回复
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 68/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 冷清
- 技术栈
- javascript, react
调研方向
从 packages/web/src/components/basic/ReactiveBase.js 的第 140-162 行开始，检查 URL 参数在进入 selectedValues 之前是如何解析的。验证解析后的值是 string 原始值或包含 value 和 category 的对象，并拒绝其他形状。完成标准是：有效参数仍能填充 Redux state，而精心构造或格式错误的结构会被忽略。
由索引模型根据 Issue 内容生成。
描述
Problem
ReactiveBase.js parses URL search parameters directly with JSON.parse and injects the result into the Redux store as initial selectedValues:
Array.from(params.keys()).forEach((key) => {
try {
const parsedParams = JSON.parse(params.get(key));
selectedValues[key] = { value: parsedParams.value || parsedParams, ... };
} catch (e) { /* silent failure */ }
});
A crafted URL can inject arbitrary object structures into the store, potentially triggering unexpected behavior in downstream components that consume selectedValues.
Location
packages/web/src/components/basic/ReactiveBase.js, lines 140-162
Suggested Fix
Validate the shape of parsed values before injection. Expected shapes are string primitives or { value, category } objects. Reject anything that doesn't match.
Severity
High — URL injection into application state
- 主要语言
- JavaScript
- 星标
- 4.9k
- 派生
- 479
- 平均合并
- 4 天 10 小时
- 30 天内合并 PR
- 4
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
appbaseio/reactivesearch 的其他 Issue
-
ci: add Node.js version matrix, caching, and lint step to test workflow可能已有人在做 @Talha12Shiekh 于 79 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 84/100
appbaseio/reactivesearch#2322 ·
维护者通常 1 天内回复
-
chore: add Dependabot config for automated dependency updates可能已有人在做 @Talha12Shiekh 于 79 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 78/100
appbaseio/reactivesearch#2321 · 1 条评论 ·
维护者通常 1 天内回复
-
refactor: consolidate 12 near-identical SVG icon components into one pattern可能已有人在做 @Talha12Shiekh 于 81 天前认领。 未关闭
难度 4/5 3-5 天 新手友好度 48/100
appbaseio/reactivesearch#2324 ·
维护者通常 1 天内回复
-
refactor: SearchBox.js is 2692 lines — extract subcomponents for maintainability可能已有人在做 @Talha12Shiekh 于 79 天前认领。 未关闭
难度 4/5 3-5 天 新手友好度 55/100
appbaseio/reactivesearch#2323 ·
维护者通常 1 天内回复
-
bug: image src attributes not validated for javascript: protocol URLs可能已有人在做 @Talha12Shiekh 于 79 天前认领。 未关闭
难度 4/5 3-5 天 新手友好度 55/100
appbaseio/reactivesearch#2320 ·
维护者通常 1 天内回复
查看 appbaseio/reactivesearch 的全部 Issue
相似的 Issue
-
难度 1/5 1 小时以内 新手友好度 65/100
521xueweihan/HelloGitHub#4025 ·
-
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 83/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
c1overzz/dsh-bili-scroll#22 ·
-
bug
难度 2/5 1-3 小时 新手友好度 70/100
maproulette/maproulette-frontend#2944 ·
维护者通常 1 天内回复