回答編集履歴
4
誤字の修正
test
CHANGED
|
@@ -18,7 +18,7 @@
|
|
|
18
18
|
|
|
19
19
|
|
|
20
20
|
|
|
21
|
-
以下、そういった危険性を承知の上、かつ、外部に公開しないような限定されたところで利用したいというニーズに対する回答とご理解ください
|
|
21
|
+
以下、そういった危険性を承知の上、かつ、外部に公開しないような限定されたところで利用したいというニーズに対する回答とご理解ください。
|
|
22
22
|
|
|
23
23
|
|
|
24
24
|
|
3
記述に誤りがある部分を修正
test
CHANGED
|
@@ -6,7 +6,7 @@
|
|
|
6
6
|
|
|
7
7
|
|
|
8
8
|
|
|
9
|
-
- [完全なSQLインジェクション対策 – yohgaki's blog]
|
|
9
|
+
- [完全なSQLインジェクション対策 – yohgaki's blog](https://blog.ohgaki.net/complete-sql-injection-counter-measure)
|
|
10
10
|
|
|
11
11
|
|
|
12
12
|
|
|
@@ -74,7 +74,7 @@
|
|
|
74
74
|
|
|
75
75
|
public Page<TaskEntity> searchmaterial(String title, String titleKana,Pageable pageable) {
|
|
76
76
|
|
|
77
|
-
return taskrepository.findByTitleAndTitleKana(
|
|
77
|
+
return taskrepository.findByTitleAndTitleKana(title, titleKana, pageable);
|
|
78
78
|
|
|
79
79
|
}
|
|
80
80
|
|
2
質問者のコメントと xebme さんのご指摘を受け、修正。
test
CHANGED
|
@@ -1,3 +1,29 @@
|
|
|
1
|
+
★★★ 注意 ★★★
|
|
2
|
+
|
|
3
|
+
|
|
4
|
+
|
|
5
|
+
xebme さんのご指摘にある通り、ユーザー入力に「%」、「_」を許すことは、SQL インジェクションの危険性があります。以下のリンクが適切かどうか、精査できていませんが、参考になれば幸いです。
|
|
6
|
+
|
|
7
|
+
|
|
8
|
+
|
|
9
|
+
- [完全なSQLインジェクション対策 – yohgaki's blog] (https://blog.ohgaki.net/complete-sql-injection-counter-measure)
|
|
10
|
+
|
|
11
|
+
|
|
12
|
+
|
|
13
|
+
> 「とにかくプレイスホルダを使おう」では脆弱性は無くなりません。
|
|
14
|
+
|
|
15
|
+
|
|
16
|
+
|
|
17
|
+
- [気になるJava 分かったつもりで終われない。 Spring Data JPA JPQLインジェクション対策](http://bookmount8.blog.fc2.com/blog-entry-43.html)
|
|
18
|
+
|
|
19
|
+
|
|
20
|
+
|
|
21
|
+
以下、そういった危険性を承知の上、かつ、外部に公開しないような限定されたところで利用したいというニーズに対する回答とご理解くださいる
|
|
22
|
+
|
|
23
|
+
|
|
24
|
+
|
|
25
|
+
【最初の回答】
|
|
26
|
+
|
|
1
27
|
メソッド名が LIKE 検索用になっていないような気が。
|
|
2
28
|
|
|
3
29
|
|
|
@@ -11,3 +37,61 @@
|
|
|
11
37
|
|
|
12
38
|
|
|
13
39
|
TaskService が呼び出しているのが、taskrepository.findAll() なので TaskRepository で追加で定義したメソッドが使われるように修正する必要がありそうです。
|
|
40
|
+
|
|
41
|
+
|
|
42
|
+
|
|
43
|
+
【追記その２】
|
|
44
|
+
|
|
45
|
+
|
|
46
|
+
|
|
47
|
+
> しかし、冒頭でアドバイスを頂いた「TaskRepository で追加で定義したメソッドが使われるように修正する必要がありそう」に関して、taskrepository.findAll() をどのように変更すれば良いかがわかりません。
|
|
48
|
+
|
|
49
|
+
|
|
50
|
+
|
|
51
|
+
以下、動作確認はしていませんが。以下のように直した場合の動作を確認してみてください。
|
|
52
|
+
|
|
53
|
+
|
|
54
|
+
|
|
55
|
+
```java
|
|
56
|
+
|
|
57
|
+
@Service
|
|
58
|
+
|
|
59
|
+
@Transactional
|
|
60
|
+
|
|
61
|
+
public class TaskService {
|
|
62
|
+
|
|
63
|
+
|
|
64
|
+
|
|
65
|
+
@Autowired
|
|
66
|
+
|
|
67
|
+
private TaskRepository taskrepository;
|
|
68
|
+
|
|
69
|
+
@Autowired
|
|
70
|
+
|
|
71
|
+
private TaskSpecification taskspecification;
|
|
72
|
+
|
|
73
|
+
|
|
74
|
+
|
|
75
|
+
public Page<TaskEntity> searchmaterial(String title, String titleKana,Pageable pageable) {
|
|
76
|
+
|
|
77
|
+
return taskrepository.findByTitleAndTitleKana(String title, String titleKana, Pageable pageable);
|
|
78
|
+
|
|
79
|
+
}
|
|
80
|
+
|
|
81
|
+
}
|
|
82
|
+
|
|
83
|
+
```
|
|
84
|
+
|
|
85
|
+
|
|
86
|
+
|
|
87
|
+
ただし、この修正だとこれまでの「%」、「_」を使わない入力における部分一致検索が使えません。
|
|
88
|
+
|
|
89
|
+
xebme さんが
|
|
90
|
+
|
|
91
|
+
|
|
92
|
+
|
|
93
|
+
> さらに、検索条件を調べて、%_があればLIKE検索、なければ通常検索にしなければならず、ここにも設計不備があると思われます。
|
|
94
|
+
|
|
95
|
+
|
|
96
|
+
|
|
97
|
+
と書かれている通り、条件分岐が必要になると思います。
|
1
TaskRepository で追加で定義したメソッドが使われていないので追記
test
CHANGED
|
@@ -3,3 +3,11 @@
|
|
|
3
3
|
|
|
4
4
|
|
|
5
5
|
[【Spring Data JPA】自動実装されるメソッドの命名ルール - Qiita](https://qiita.com/shindo_ryo/items/af7d12be264c2cc4b252)あたりとか、参考にならないでしょうか。
|
|
6
|
+
|
|
7
|
+
|
|
8
|
+
|
|
9
|
+
【追記】
|
|
10
|
+
|
|
11
|
+
|
|
12
|
+
|
|
13
|
+
TaskService が呼び出しているのが、taskrepository.findAll() なので TaskRepository で追加で定義したメソッドが使われるように修正する必要がありそうです。
|