StackStorm/st2

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

Open

#4.789 geöffnet am 12. Sept. 2019

Auf GitHub ansehen
 (18 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (740 Forks)batch import
CLIbugcomponent:st2clientgood first issue

Repository-Metriken

Stars
 (5.794 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche, wie die st2 Schlüssel Wert Speicher API mit URL Codierung umgeht, insbesondere für Schlüssel mit Sonderzeichen wie '/'. Sieh dir den Keys Endpunkt im st2api Controller (wahrscheinlich in st2api/controllers/v1/keys.py) und die zugehörige Middleware an. Das Problem ist, dass der HTTP Client den Schlüssel codiert, was zu einem 404 führt, weil die Route nicht übereinstimmt. Eine Lösung könnte entweder die URL Decodierung des Schlüssels im Routen Handler oder das Ablehnen solcher Schlüssel mit einer klaren Fehlermeldung beinhalten. Überprüfe die Kommentare auf bestehende Diskussionen und erwäge Tests mit einem Schlüssel, der ein codiertes '/' enthält.
Tech Stack
pythonrest api
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
PythonREST API basicsURL encoding
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55