ok I see that rejecting without anything is what we're doing in the code for "cancel" so it's fine. Just add a comment explaining this, that having no error means the promise was rejected but that it's not an error.