【问题标题】:Error: UnhandledPromiseRejectionWarning: Unhandled promise rejection错误:UnhandledPromiseRejectionWarning:未处理的承诺拒绝
【发布时间】:2019-10-27 13:05:26
【问题描述】:

我正在使用 nodejs、express 和 argon2。我遇到了这个错误,我不知道为什么。我没有任何回调,所以我对为什么会收到此错误感到困惑。这是我的代码:

const express = require('express');
const app = express();
const port = 8080;
const argon2 = require("argon2themax");

app.use(express.json()); // for parsing application/json
app.use(express.urlencoded({ extended: true })); // for parsing application/x-www-form-urlencoded

// dumby
global.user = [{username:"u1", password:"hidden", email:"email@gmail.com"}];

// signin routes
app.post('/signup', function (req, res) {

  //console.log(req.body);

  // validate the json
  if(req.body.username === undefined || req.body.password === undefined) {
    res.status(500);
    res.render('error', { error: err });
  } else {
    try {

      // let argon2 generate our salt
      argon2.generateSalt()
      .then(function(salt) {

          // Hashing happens in an asynchronous event using libuv so your system can
          // still process other IO items in the Node.JS queue, such as web requests.
          return argon2.hash(req.body.password, salt, maxOpts);

      }).then(function(hash) {

          // This hash is what you should store in your database. Treat it as an opaque string.
          global.user.push({username:req.body.username, password:hash});

          // Verifying the hash against your user's password is simple.
          res.json(true);
      })
    } catch(error) {
      console.log("Unexpected error: " + error);
      //res.status(500);
      //res.render('error', { error: err });
    }
  }
})

app.post('/signin', function (req, res) {

  var success = false;

  global.user.forEach(function(item, index, array) {
    if(item.username == req.body.username) {
      try {
        argon2.verify(item.password, req.body.password).then(function(match) {
              if(match) {
                console.log("pword match");
                res.json(true);
              } else {
                res.json(false);
              }
              console.log("done");
          });
      } catch(err) {
        console.log(err);
      }
    } else {
      console.log("no uname");
      res.json(false);
    }
  });

})

只有当我在 /signin 路由之前点击 /signup 路由时,我的服务器才会抛出此错误。我相信这是一个异步问题,但我不知道如何解决它。完整的错误信息如下:

(node:21623) UnhandledPromiseRejectionWarning: Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
    at ServerResponse.setHeader (_http_outgoing.js:470:11)
    at ServerResponse.header (/Users/[redacted]/Public/Docs/[redacted]/node_modules/express/lib/response.js:775:10)
    at ServerResponse.send (/Users/[redacted]/Public/Docs/[redacted]/node_modules/express/lib/response.js:174:12)
    at ServerResponse.json (/Users/[redacted]/Public/Docs/[redacted]/node_modules/express/lib/response.js:271:15)
    at /Users/[redacted]/Public/Docs/[redacted]/server.js:106:17
(node:21623) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:21623) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

【问题讨论】:

  • 这通常意味着你在同一个代码路径中调用了两次res.status。
  • 我根本不调用 res.status。我猜这是调用 res.status 的快递,但我不确定如何或在哪里
  • 在放入调试语句后,我发现这是一个异步问题。线程退出 /signin 处理程序,等待 argon2.verify() 完成,然后返回该分支。我该如何解决这个问题?
  • @VikashSingh 最初我也这么认为,但后来发现这只是症状。真正的问题如下所述。

标签: javascript node.js express ecmascript-6 es6-promise


【解决方案1】:

对于这两种方法,这不是为 Promise 链实现 catch 的正确方法:

try {

      // let argon2 generate our salt
      argon2.generateSalt()
      .then(function(salt) {

          // Hashing happens in an asynchronous event using libuv so your system can
          // still process other IO items in the Node.JS queue, such as web requests.
          return argon2.hash(req.body.password, salt, maxOpts);

      }).then(function(hash) {

          // This hash is what you should store in your database. Treat it as an opaque string.
          global.user.push({username:req.body.username, password:hash});

          // Verifying the hash against your user's password is simple.
          res.json(true);
      })
    } catch(error) {
      console.log("Unexpected error: " + error);
      //res.status(500);
      //res.render('error', { error: err });
    }

相反,您应该执行以下操作:

  // let argon2 generate our salt
  argon2.generateSalt()
  .then(function(salt) {

      // Hashing happens in an asynchronous event using libuv so your system can
      // still process other IO items in the Node.JS queue, such as web requests.
      return argon2.hash(req.body.password, salt, maxOpts);

  }).then(function(hash) {

      // This hash is what you should store in your database. Treat it as an opaque string.
      global.user.push({username:req.body.username, password:hash});

      // Verifying the hash against your user's password is simple.
      res.json(true);
  }).catch(error => {
       console.log("Unexpected error: " + error);
  });

【讨论】:

  • 这并没有解决问题,但是 .catch() 和 try-catch 有什么区别?
  • 我只是想了一遍,我意识到这可能会解决问题,但我以不同的方式解决了它
  • @JustinCase 关于您的第一条评论:用 try/catch 包围代码将无济于事,因为此代码返回未实现的承诺,因此在执行退出 try 子句之前不会“抛出”任何内容。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2019-04-16
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多