Sanitizing input text for "get" queries?
还没有人认领这个 Issue。
评估
调研方向
使用 /study/1/samples 端点和 issue 中显示的 term 值重现该故障。跟踪该 GET 查询的服务器端处理程序，并确定带引号的输入如何到达发生故障的操作；完成的标准是，对于此 specimen ID，请求不再返回 500，并且预期的查询行为已由回归测试覆盖。
由索引模型根据 Issue 内容生成。
描述
Doesn't look like this has been logged as an issue before, so I'm documenting it here.
In the style of xkcd, I tried plating a specimen with the ID ');console.log('hi');. Fortunately, this doesn't result in hi being printed to the console; unfortunately, it does cause a 500 internal server error:
GET https://127.0.0.1:8181/study/1/samples?term=%27);console.log(%27hi%27); 500 (Internal Server Error)
Looks like it's the ' character (which gets converted to %27) that's problematic.
I don't imagine this is a common use case, but it might be worth addressing?
Edit: also, if this behavior is expected (i.e. the server throws an error if it detects a wacky specimen ID like ');console.log('hi');) then sorry, and feel free to close this.
- 主要语言
- Python
- 星标
- 2
- 派生
- 15
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
我们还没有检查这个项目的环境配置文件。先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
biocore/LabControl 的其他 Issue
-
bug front-end question
难度 4/5 3-5 天 新手友好度 30/100
biocore/LabControl#594 ·
-
front-end question
难度 5/5 一周以上 新手友好度 35/100
biocore/LabControl#593 ·
-
Cache list of active samples when the active study is changed in the plating interface可能重新可做 @fedarko 于 2575 天前认领，目前没有进行中的 PR。 未关闭front-end
biocore/LabControl#592 · 已指派 1 人 ·
-
code refactor front-end
难度 5/5 一周以上 新手友好度 25/100
biocore/LabControl#591 ·
-
priority:low
难度 3/5 1-2 天 新手友好度 20/100
biocore/LabControl#590 ·
查看 biocore/LabControl 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 88/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
spec-kitty/spec-kitty#5319 ·
维护者通常 1 天内回复
-
backend::vllm diffusion multimodal
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
openai/openai-agents-python#5229 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
维护者通常 1 天内回复