ClickHouse / ClickHouse/clickhouse-java

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

Ouverte
#2,739 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
area:jdbc-metadata jdbc-v2
Langage dominant
Java
Étoiles
1.6k
Forks
636
Merge moyen
2 j 23 h
PR mergées (30 j)
29

Description

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

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par localiser DatabaseMetaDataImpl et recenser ses requêtes construites dynamiquement. Examinez chaque requête pour la convertir en PreparedStatement et identifiez les cas où la conversion n’est pas possible et où l’échappement doit être traité. La tâche est terminée lorsque toutes les requêtes concernées utilisent des PreparedStatement et que les cas restants disposent de l’échappement nécessaire.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
databases, security
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.