Say I have a promise called myProm, and say I have success and error handlers called onSuccess and onError.

Whenever my promise takes longer than 10 seconds to plete, I want a function called timeoutHandler to be executed, but if that happens, neither onSuccess nor onError should be executed. (Similarly, if either onSuccess or onError runs, I don't want my timeoutHandler to be executed.)

I've e up with the following snippet for this.

new Promise((suc, err) => {
    let outOfTime = false;
    const timeoutId = window.setTimeout(() => {
        outOfTime = true;
        timeoutHandler();
    }, 10000);
    myProm.then(
        (...args) => {
            if (!outOfTime) {
                window.clearTimeout(timeoutId);
                suc(...args);
            }
        },
        (...args) => {
            if (!outOfTime) {
                window.clearTimeout(timeoutId);
                err(...args);
            }
        }
    );
}).then(onSuccess, onError);

However, in case of a timeout, my newly defined promise will be forever-pending. Could this have any negative side effects? For example, the runtime not being able to clean up the Promise object because it's still pending (or something along those lines).

Say I have a promise called myProm, and say I have success and error handlers called onSuccess and onError.

Whenever my promise takes longer than 10 seconds to plete, I want a function called timeoutHandler to be executed, but if that happens, neither onSuccess nor onError should be executed. (Similarly, if either onSuccess or onError runs, I don't want my timeoutHandler to be executed.)

I've e up with the following snippet for this.

new Promise((suc, err) => {
    let outOfTime = false;
    const timeoutId = window.setTimeout(() => {
        outOfTime = true;
        timeoutHandler();
    }, 10000);
    myProm.then(
        (...args) => {
            if (!outOfTime) {
                window.clearTimeout(timeoutId);
                suc(...args);
            }
        },
        (...args) => {
            if (!outOfTime) {
                window.clearTimeout(timeoutId);
                err(...args);
            }
        }
    );
}).then(onSuccess, onError);

However, in case of a timeout, my newly defined promise will be forever-pending. Could this have any negative side effects? For example, the runtime not being able to clean up the Promise object because it's still pending (or something along those lines).

• javascript
• promise
• ecmascript-6
• es6-promise

• 2 Related stackoverflow./questions/20068467/… – Benjamin Gruenbaum Commented Jul 10, 2016 at 12:06

2 Answers 2

There should be no side effect. It would be a browser bug if a non-referenced Promise in whatever state is keeping resources.

Just make sure you don't keep any reference to the Promise object and you'll be fine.

Beware that certain APIs such as setTimeout will keep a reference to the closure up to the timeout value. This means that if you have a long timeout, like the 10s one, you should clear it as soon as you don't need it anymore. Otherwise your code can call thousands of setTimeout within 10s, and each of them will keep a reference to the closure, which in your case will reference the Promise.

You can use Promise.race(), set timeoutHandler as a function which returns a rejected a Promise in ten seconds, else onSuccess should be called at fulfilled Promise of myProm

function myProm() {
  return new Promise((success, err) => {
    setTimeout(() => {
      success("myProm")
    }, Math.floor(Math.random() * 11000))
  })
}

function timeoutHandler() {
  return new Promise((_, timeout) => {
    setTimeout(() => {
      timeout(new Error("timeoutHandler"));
    }, 10000)
  })
}

function onSuccess(data) {
  console.log("success", data)
}

function onError(err) {
  console.log("err:", err)
}

function onTimeout(e) {
  if (e.message && e.message === "timeoutHandler") {
    console.log(e.message + " handled");
  }
  else { 
    onError(e)
  }
}

Promise.race([myProm(), timeoutHandler()])
.then(onSuccess, onTimeout);

plnkr http://plnkr.co/edit/9UD5syOEOc1oQGdRTRRm?p=preview