StackStorm/st2

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

Open

#4.789 aberto em 12 de set. de 2019

Ver no GitHub
 (18 comments) (0 reactions) (0 assignees)Python (740 forks)batch import
CLIbugcomponent:st2clientgood first issue

Métricas do repositório

Stars
 (5.794 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Investigue como a API de armazenamento de chave valor do st2 lida com a codificação de URL, especialmente para chaves com caracteres especiais como '/'. Examine o endpoint de chaves no controlador st2api (provavelmente em st2api/controllers/v1/keys.py) e o middleware relacionado. O problema é que o cliente HTTP codifica a chave, causando um 404 porque a rota não corresponde. Uma correção pode envolver decodificar a chave no manipulador da rota ou rejeitar tais chaves com um erro claro. Verifique os comentários para discussões existentes e considere testar com uma chave que inclua '/' codificado.
Pilha de tecnologia
pythonrest api
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
PythonREST API basicsURL encoding
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
55