StackStorm/st2

st2 fails to store a Key-Value in the datastore if there is a "/" in the key name.

Open

#4,789 创建于 2019年9月12日

在 GitHub 查看
 (18 评论) (0 反应) (0 负责人)Python (5,794 star) (740 fork)batch import
CLIbugcomponent:st2clientgood first issue

描述

SUMMARY

st2 fails to store a Key-Value in the datastore if there is a "/" in the key name. I am sorry if I missed anything in the docs, which says that you cannot store a key with "/" in it and please feel free to close this issue if that is the case. Thank you.

STACKSTORM VERSION

st2 3.1.0, on Python 2.7.5

OS, environment, install method:

CentOS/Docker, custom install.

Steps to reproduce the problem

bash-4.2$ st2 key set "foo / bar" "foo_bar"
ERROR: 404 Client Error: Not Found
MESSAGE: The resource could not be found. for url: http://127.0.0.1:9101/v1/keys/foo%20/%20bar

Expected Results

st2 should have accepted the key-value pair and stored it.

Actual Results

st2 has failed to store the key-value pair.

贡献者指南

技术栈
pythonrest api
领域
backend
议题类型
bug
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 hours
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
stale
清晰度议题是否清楚说明期望改动、验收标准和下一步。
mostly clear
前置要求
PythonREST API basicsURL encoding
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
55
研究方向
Investigate how the st2 key value store API handles URL encoding, especially for keys with special characters like '/'. Look at the st2api controller for keys endpoint (likely in st2api/controllers/v1/keys.py) and related middleware. The issue is that the HTTP client encodes the key, causing a 404 because the route doesn't match. A fix could involve either URL decoding the key in the route handler or rejecting such keys with a clear error. Check comments for existing discussion and consider testing with a key that includes encoded '/'.