apache/gravitino

[Improvement] Possible SQL injection in MysqlDatabaseOperations.java and JdbcDatabaseOperations.java

Open

#3.026 geöffnet am 19. Apr. 2024

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Java (887 Forks)auto 404
good first issueimprovement

Repository-Metriken

Stars
 (3.058 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

What would you like to be improved?

The database name should be validated before but used to construct an SQL to show tables. (Search for "SHOW TABLES IN".) Note that the SQL strings are also built in different ways in each file and probably should be built the same way.

How should we improve?

Validate the name or use query parameters.

Contributor Guide

Research-Richtung
Validieren Sie den Datenbanknamen, bevor Sie SQL Abfragen erstellen, oder verwenden Sie Abfrageparameter, um SQL Injection zu verhindern. Überprüfen Sie die Methoden, die SHOW TABLES IN Anweisungen in MysqlDatabaseOperations.java und JdbcDatabaseOperations.java erstellen, und stellen Sie konsistente parametrisierte Abfragen sicher.
Tech Stack
java
Domain
backend
Issue Type
Sicherheit
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.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaSQL
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60