elastic/kibana

[Management] Number fields accept invalid input

Offen

#197.506 geöffnet am 23.10.2024

Auf GitHub ansehenContributor Guide
 (5 Kommentare) (0 Reaktionen) (2 zugewiesene Personen)TypeScript (8.021 Forks)batch import
Team:Kibana Managementbuggood first issue

Repository-Metriken

Stars
 (19.065 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

While testing https://github.com/elastic/kibana/pull/196527, I noticed that different form fields in Kibana Management plugins accept invalid input in number fields. Examples I noticed:

  • Ingest pipelines creation flow - add processor form
  • ILM creation flow

https://github.com/user-attachments/assets/6a883a7b-0cf4-43e2-bc36-c0ff6d2ebf1b

https://github.com/user-attachments/assets/45e21ff4-fc0d-4ef0-89c9-4dc922e1abfe

I think the e, -, and , characters are allowed by the Eui number field itself and it contains some validation for valid numbers. We should probably add validations from our side for valid numbers as well and not allow submitting the form if the number is invalid.

Contributor Guide

Research-Richtung
Untersuchen Sie die Validierung von Zahleneingaben in den Verwaltungsformularen von Kibana. Identifizieren Sie, wo EuiFieldNumber ohne ordnungsgemäße Validierung verwendet wird, und fügen Sie clientseitige Validierung hinzu, um das Senden ungültiger Zahlen (z. B. Buchstaben wie 'e', Minuszeichen, Kommas) zu verhindern.
Tech Stack
typescript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
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.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitReactKibana
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70