Node / JS承诺被阻止

Question

New to Node/JS and I'm creating a password recovery page for an exiting IT portal, it searches AD(ldap) and a DB where the user has registered. Based on the results from both presents user with options to auth then change pwd

When I make a request ($get()) from the client to find a user the backend stalls intermittently. See console logging below

This is the console logging

Attempting to find : <account name>
Found AD user: <account name>
in acc prep
in sql request
in getUserRegistration //within auth controller

.
.here it prints the results of the sql query // At this point I would expect the promise to be fulfilled.
.

This is where time is spent doing who knows what. It eventually completes but usually takes minutes. then prints...

in sql results

The stall seems to be after calling return auth.getUserRegistration(globalUser.sql) from within a then(). The logging from the auth controller very quickly logs the results from the sql query which is right before it should be resolved, thereby returning the results to the following then(). So i don't believe there to be a issue with the auth controller (which independently works flawlessly) but rather within the resolve(). Perhaps because its within a another promise?.

What I find Interesting and confuses me is if I browse to the portal from another tab/browser the backend work completes and the page renders immediately.

See code below

/* Client request */

$.get(urlqry, function(result) {
  if (result.match("<div id=\"pop\">")) {

    $.modal(result, {
        onClose: function () {
          $("[id^=simplemodal").remove();
          $("#findload").hide();
          $("#forgetbtn").show();
          $.modal.close();
      }
    });

  } else {
    console.log("in else");
    $("#divid").html(result)
  }

});

/* 
this is the route that renders based the work from 2 different controllers
1. check if user exists in Active Directory (ldap)
2. check if user has registered (sql)
*/

router.get('/forgetpwd', function(req, res) {

  var usrname = req.query.usr
  var domain = req.query.realm

  if (usrname && domain) {

    console.log("Attempting to find : " + usrname + "@" + domain);

    var globalUser = {
      username: "",
      upn: "",
      accounts: [],
      sql: {},
      registered: false
    }

    ad.getUser(usrname, domain)

    .then(function(result) {
      if (result.userPrincipalName) {

        console.log("Found AD user: " + result.userPrincipalName);

        globalUser.username = result.sAMAccountName
        globalUser.upn = result.userPrincipalName
        globalUser.accounts = result.extensionAttribute15

      } else {

        var fail = {
          status: "fail",
          message: "Unknown Error"
        }

        var msg = result ? result : fail

        throw msg;

      }

    })

    .then(function(result) {
        console.log("in acc prep");

        globalUser.sql.username = globalUser.username
        var upns = globalUser.accounts

        if (upns) {
          upns = upns.split(",")

          for (u in upns) {

            var upn = upns[u].split("@")
            var domain = upn.slice(-1)[0].toLowerCase()

            switch (domain) {
              case "domain1":
                globalUser.sql.domain1UPN = upns[u]
                break;
              case "domain2":
                globalUser.sql.domain2UPN = upns[u]
                break;
              case "domain3":
                globalUser.sql.ihUPN = upns[u]
                break;
              default:
                globalUser.accounts = ""
                break;
            }
          }
        } else {
          globalUser.accounts = ""
        }
        return;

      })

    .then(function(result) {
      console.log("in sql request");

      return auth.getUserRegistration(globalUser.sql)

    })

    .then(function(result) {

      console.log("in sql results");

      if (result.length > 0) {

        globalUser.registered = true
        return

      } else {

        globalUser.registered = false
        return

      }

    })

    .then(function(result) {

      var accounts = {
        username: globalUser.username,
        accounts: globalUser.accounts.split(","),
        registered: globalUser.registered

      }

      console.log(accounts);
      res.render('./forget-options', accounts);

    })

    .catch(function(error) {
      console.log("in catch");
      var failure = {
        status: "fail",
        message: error

      }
      console.log(failure);
      res.render("./ack", failure);
      return

    })

  } else {

    console.log("User field empty and/or Domaim no selected");

  }
});

/*
Checks if a a record exist in a DB based on results from ad.getUser
requires edge/edge-sql
*/

auth.getUserRegistration = function(accounts) {
  return new Promise(function(resolve, reject) {

    var query = "select * from [users] where (username = "
    query += "\'" + accounts.username + "\'"
    if (accounts.domain2UPN) {
      query += " or domain2UPN = \'" + accounts.domain2UPN + "\'"
    }
    if (accounts.domain1UPN) {
      query += " or domain1UPN = \'" + accounts.domain1UPN + "\'"
    }

    query += ") and q1col is not null and q2col is not null and pin is not null;"

    var sqlparams = {
      connectionString: "Data Source=name.domain.com;Initial Catalog=dbname;Integrated Security=True",
      source: query
    };

    var getReg = edge.func('sql', sqlparams);

    getReg(null, function(error, result) {

      if (error) {

        console.log("Retrieving user ?? registration");
        reject(error);

      } else {

        console.log("in getUserRegistration");
        console.log(result);
        resolve(result);

      }
    });
  });
}

Answer 1

I ended up changing my db module from edge-sql to mssql and updated my code accordingly. It worked immediately.

auth.getUserRegistration = function(accounts) {
  return new Promise(function(resolve, reject) {

    var query = "select * from [users] where (username = "
    query += "\'" + accounts.username + "\'"
    if (accounts.domain2UPN) {
      query += " or domain2UPN = \'" + accounts.domain2UPN + "\'"
    }
    if (accounts.domain1UPN) {
      query += " or domain1UPN = \'" + accounts.domain1UPN + "\'"
    }

    query += ") and q1col is not null and q2col is not null and pin is not null;"

    sql.connect("mssql://<usr>:<pwd>@<server>/<db>").then(function() {         
        new sql.Request().query(query).then(function(recordset) {
            console.log(recordset);
            resolve(recordset);
        }).catch(function(err) {
            console.log(err);
        });

    }).catch(function(err) {
            console.log(err);
        });
    });
  });
}