ClickHouse / ClickHouse/clickhouse-java
[JDBC-V2] Make `DatabaseMetaDataImpl` use `PreparedStatement`.
- 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
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