bcryptでパスワードをhash化する際のsqlのエラーが直せない。

Question

現在、新規登録機能を作っており、パスワードのhash化をするためにプログラムを書いたら

throw err; // Rethrow non-MySQL errors
^

TypeError: Cannot read properties of undefined (reading 'insertId')

というエラーが出て、先に進めなくなってしまっています。
おそらく対象のコードは以下になります。

app.post("/signup",

 (req, res, next) => {
   console.log("入力値の空チェック");

   const username = req.body.username;
   const email = req.body.email;
   const password = req.body.password;

   const errors = [];

   if(username === ""){
     errors.push("ユーザー名が空です");
   }
   if(email === ""){
     errors.push("メールアドレスが空です");
   }
   if(password === ""){
     errors.push("パスワードが空です");
   }

   console.log(errors);

   if(errors.length > 0){
     res.render("signup.ejs", {errors:errors});
   }else{
     next();
   }
 },
 (req, res, next) => {
   console.log("メールアドレスの重複チェック");
   const email = req.body.email;
   const errors = [];
   connection.query(
      'SELECT * FROM users WHERE email = ?',
      [email],
      (error, results) => {
        if (results.length > 0) {
          errors.push("ユーザー登録に失敗しました");
          res.render("signup.ejs", {errors:errors});
        } else {
          next();
        }
      }
    );
 },
 (req, res) => {
   console.log('ユーザー登録');
     const username = req.body.username;
     const email = req.body.email;
     const password = req.body.password;
     bcrypt.hash(password, 10, (error, hash) => {
       connection.query(
       'INSERT INTO users (username, email, password) VALUES (?, ?, ?)',
       [username, email, hash],
       (error, results) => {
         req.session.userId = results.insertId;
         req.session.username = username;
         res.redirect('/');
       }
     );
     })
   }
 );

どなたか教えていただけませんでしょうか？

Comments

[wasd0109]

req.session.userId = results.insertId;のところでresultsがundefinedになってることによるエラーですね
errorのところでなんかエラーメッセージとか出てたりしてますか？

[josikikusokurae]

at Query.<anonymous> (/Users/.../Desktop/list-app/app.js:114:39)
at Query.<anonymous> (/Users/.../Desktop/list-app/node_modules/mysql/lib/Connection.js:526:10)
at Query._callback (/Users/.../Desktop/list-app/node_modules/mysql/lib/Connection.js:488:16)
at Query.Sequence.end (/Users/.../Desktop/list-app/node_modules/mysql/lib/protocol/sequences/Sequence.js:83:24)
at Query.ErrorPacket (/Users/.../Desktop/list-app/node_modules/mysql/lib/protocol/sequences/Query.js:92:8)
at Protocol._parsePacket (/Users/.../Desktop/list-app/node_modules/mysql/lib/protocol/Protocol.js:291:23)
at Parser._parsePacket (/Users/.../Desktop/list-app/node_modules/mysql/lib/protocol/Parser.js:433:10)
at Parser.write (/Users/.../Desktop/list-app/node_modules/mysql/lib/protocol/Parser.js:43:10)
at Protocol.write (/Users/.../Desktop/list-app/node_modules/mysql/lib/protocol/Protocol.js:38:16)
at Socket.<anonymous> (/Users/.../Desktop/list-app/node_modules/mysql/lib/Connection.js:88:28)
こんな感じのが出てきました。

[wasd0109]

(error, results) => {
req.session.userId = results.insertId;
req.session.username = username;
res.redirect('/');
}
のところにconsole.log(error)の出力も見せてもらえますか？

[josikikusokurae]

すみません。知識が浅いもので、何をすればよいでしょうか？

[wasd0109]

ちょっとわかりずらくて申し訳ないです。最後ユーザーをデーてベースに保存するときのコールバック関数
req.session.userId = results.insertId;の一行上にconsole.log(error);を入れて一回プログラム執行した後そのconsole.logの出力見たいです

[josikikusokurae]

Error: ER_DATA_TOO_LONG: Data too long for column 'password' at row 1

code: 'ER_DATA_TOO_LONG',
errno: 1406,
sqlMessage: "Data too long for column 'password' at row 1",
sqlState: '22001',
index: 0,
sql: "INSERT INTO users (username, email, password) VALUES ('daisuke', 'daisuke@example', '$2b$10$LQ3Nies8etOpZ62PQ9HlzOUyaUx2KipVoKQb5hJ3or5d4GUn1sqmO')"

と出てきました！

[wasd0109]

なるほど
Hash化された後のパスワードが長すぎてデータベースのpasswordフィールドに設定されたデータタイプの上限を超えたようですね

ネットで調べてみたらbcryptで生成されるhashのながさは60文字なので、今passwordフィールドに設定したデータタイプを60文字入れるように設定してみてください

[josikikusokurae]

できました。ありがとうございました！

Answer 1

passwordフィールドの上限を再設定すると解決できました。