diff options
author | Folker Bernitt <fbernitt@thoughtworks.com> | 2015-10-29 10:18:54 +0100 |
---|---|---|
committer | Folker Bernitt <fbernitt@thoughtworks.com> | 2015-10-29 10:20:14 +0100 |
commit | cb405e32d1a66cb047aca44c81203a58cc5c2068 (patch) | |
tree | 0e5a00b03672df8d9198a566e23b192b5f07d48b /web-ui/app/js/helpers/monitored_ajax.js | |
parent | eb3b4bbbd3c5e3c842f9d12622bb25b45b97b38c (diff) |
Redirect user to login if ajax request fails with 302
- See project issue 162
- Should solve 'disconnected' problems
Diffstat (limited to 'web-ui/app/js/helpers/monitored_ajax.js')
-rw-r--r-- | web-ui/app/js/helpers/monitored_ajax.js | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/web-ui/app/js/helpers/monitored_ajax.js b/web-ui/app/js/helpers/monitored_ajax.js index 7f9a9beb..d05fdc4c 100644 --- a/web-ui/app/js/helpers/monitored_ajax.js +++ b/web-ui/app/js/helpers/monitored_ajax.js @@ -15,7 +15,7 @@ * along with Pixelated. If not, see <http://www.gnu.org/licenses/>. */ -define(['page/events', 'views/i18n'], function (events, i18n) { +define(['page/events', 'views/i18n', 'helpers/browser'], function (events, i18n, browser) { 'use strict'; @@ -45,11 +45,17 @@ define(['page/events', 'views/i18n'], function (events, i18n) { return $.ajax(url, config).fail(function (xmlhttprequest, textstatus, message) { if (!config.skipErrorMessage) { - var msg = (xmlhttprequest.responseJSON && xmlhttprequest.responseJSON.message) || + var msg = (xmlhttprequest.responseJSON && xmlhttprequest.responseJSON.message) || messages[textstatus] || 'unexpected problem while talking to server'; - on.trigger(document, events.ui.userAlerts.displayMessage, { message: i18n(msg) }); + on.trigger(document, events.ui.userAlerts.displayMessage, {message: i18n(msg)}); } + + if (xmlhttprequest.status === 302) { + var redirectUrl = xmlhttprequest.getResponseHeader('Location'); + browser.redirect(redirectUrl); + } + }.bind(this)); } |