ClickHouse / ClickHouse/clickhouse-java

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

Abierto
#2,739 0 comentarios 0 reacciones 0 asignados Ver en GitHub
area:jdbc-metadata jdbc-v2
Lenguaje dominante
Java
Estrellas
1.6k
Forks
636
Merge medio
2 d 23 h
PR fusionados (30 d)
29

Descripción

## 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

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Empieza localizando DatabaseMetaDataImpl y haciendo un inventario de sus consultas construidas dinámicamente. Revisa cada consulta para convertirla a PreparedStatement e identifica los casos en los que la conversión no sea posible y haya que abordar el escaping. La tarea estará terminada cuando todas las consultas aplicables utilicen PreparedStatement y los casos restantes tengan el escaping necesario.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
databases, security
Tipo de issue
Refactorización
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.