From 998ac74d32b92cd52cb85d85354451dc5fc10260 Mon Sep 17 00:00:00 2001 From: Girish Ramakrishnan Date: Tue, 15 Sep 2015 18:08:27 -0700 Subject: [PATCH] oldConfig.location can be null If we had an update, location is not part of oldConfig. if we now do an infra update, location is undefined. --- src/apptask.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/apptask.js b/src/apptask.js index 6e25a0843..490fec1c1 100644 --- a/src/apptask.js +++ b/src/apptask.js @@ -684,8 +684,8 @@ function configure(app, callback) { stopApp.bind(null, app), deleteContainer.bind(null, app), function (next) { - // oldConfig can be null during an infra update - if (!app.oldConfig || app.oldConfig.location === app.location) return next(); + // oldConfig can be null during an infra update. location can be null when infra updated for an updated app + if (!app.oldConfig || !app.oldConfig.location || app.oldConfig.location === app.location) return next(); unregisterSubdomain(app, app.oldConfig.location, next); }, removeOAuthProxyCredentials.bind(null, app),