ClickHouse / ClickHouse/clickhouse-java

[JDBC-V2] Make `DatabaseMetaDataImpl` use `PreparedStatement`.

Offen
#2,739 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
area:jdbc-metadata jdbc-v2
Vorherrschende Sprache
Java
Sterne
1.6k
Forks
636
Ø Merge
2 T. 23 Std.
Gemergte PRs (30 T.)
29

Beschreibung

## Description

There are many places in `DatabaseMetaDataImpl` that construct a dynamic query. It is wrong because leads to a security issue.

- All queries should use `PreparedStatement`
- Where is it no possible - should be reviewed and needed escaping should be added

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Start by locating DatabaseMetaDataImpl and inventorying its dynamically constructed queries. Review each query for conversion to PreparedStatement, and identify cases where conversion is not possible and escaping must be addressed. Done means all applicable queries use prepared statements and the remaining cases have the needed escaping.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
databases, security
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.