ClickHouse / ClickHouse/clickhouse-java
[JDBC-V2] Make `DatabaseMetaDataImpl` use `PreparedStatement`.
オープン
area:jdbc-metadata
jdbc-v2
- 主要言語
- Java
- スター
- 1.6k
- フォーク
- 636
- 平均マージ
- 2日 23時間
- マージ済み PR(30日)
- 29
説明
## 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
コントリビューションガイド
調査の方向性
まず DatabaseMetaDataImpl を見つけ、その中で動的に構築されているクエリを洗い出します。各クエリを PreparedStatement に変換できるか確認し、変換できずエスケープ処理が必要なケースを特定します。適用可能なすべてのクエリが PreparedStatement を使用し、残りのケースに必要なエスケープ処理が実装されていれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- java
- 領域
- databases, security
- issue の種類
- リファクタリング
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 45/100