暗号化 DAO メソッド化

Question

DAO
public UserBeans findByLoginlofo(String login_Id, String password) {

Connection conn = null;
try {
conn = DataBase.getConnection();
String sql = "select * from user where login_id = ? and password = ?";

seacret( password);
PreparedStatement pStmt = conn.prepareStatement(sql);
pStmt.setString(1, login_Id);
pStmt.setString(2, password);


ResultSet rs = pStmt.executeQuery();


if (!rs.next()) {
return null;



}
String loginidData = rs.getString("login_Id");
String nameData = rs.getString("name");


return new UserBeans(loginidData, nameData);

} catch (SQLException e) {

e.printStackTrace();
} finally {

if (conn != null) {
try {
conn.close();
} catch (SQLException e) {
e.printStackTrace();

}
}
}

return null;

}

public String seacret(String password) {

String source = password;


Charset charset = StandardCharsets.UTF_8;

String algorithm = "MD5";


byte[] bytes = null;
try {
bytes =
MessageDigest.getInstance(algorithm).digest(source.getBytes(charset));
} catch (NoSuchAlgorithmException e) {

e.printStackTrace();
}
String result = DatatypeConverter.printHexBinary(bytes);


return result;
}

暗号化のメソッド

seacret( password);
PreparedStatement pStmt = conn.prepareStatement(sql);
pStmt.setString(1, login_Id);
pStmt.setString(2, password);
所がjavaのメソッド？の範囲だと思うんですけど上手く書き方が分からないです
暗号化のメソッドを使って書こうとおもっているんですが
暗号化はどのように書けばいいか分からないです

よろしければどなたか教えて頂ければ幸いです
お願いします

Comments

[m.ts10806]

分からないが多すぎて結局何をしたいのか分かりません。要件を具体的に記載し、現在のコードがどこまでできているのか、問題点なども追記してください。
コードもインデント全くないので読みづらいですし、お手元のコードをきちんとフォーマットかけて再度ご提示ください。

Answer 1

前提１：暗号化メソッドの中身一切読んでないので意図したものが返ってくるかどうかは知りません。
前提２：パスワードは復号できる形式じゃマズいのでは。
[セキュリティ] ハッシュ化と暗号化は違いますよ！！

コードのお話ですが、seacret()の戻り値にStringを設定しているようですが
DAO側で戻り値を受け取っていないため、意味がない記述になっています。
伴ってPreparedStatementに設定するのは
DAOに渡されたpasswordではなく、seacret()にpasswordを渡した戻り値です。

Comments

[takasikoarkaslf]

ご回答ありがとうございます
復号出来る形というものあまり理解していませんでした

コードはpStmt.setString(2,saecret(password));で暗号化のメソッドとパスワードを繋げる？事は出来きますか？
java勉強始めたばかりで表現の仕方が間違っていると思いますがよろしくお願いします

[tetutetu]

そういった書き方でも実行に支障は出ませんよ。
タイポは直しましょう。（そもそものスペルミスに関しては本筋から逸れるので無視）
学習目的で作成しているのであれば、後で読み返して理解ができる記述であれば問題ないかなと思います。

[takasikoarkaslf]

pStmt.setString(2,seacret(password));
にした所ログインできなくなってしまいました
デバックしてカーソルを合わせたらちゃんと値は入っています
原因はどこにあると思いますか？
あと暗号化の記述というのはDAOだけでよろいでしょうか？
サーブレットやbeansは暗号化の処理はしなくてよいですか？

[tetutetu]

＞原因はどこにあると思いますか？
元のコードではログインできていたのであれば
そもそもDBに生のパスワードで登録されているのではないでしょうか。