Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-ui/pull/30#discussion_r73879185
--- Diff: src/main/webapp/assets/js/util/brooklyn-utils.js ---
@@ -175,17 +175,22 @@ define([
};
Util.logout = function logout() {
- $.ajax({
- type: "POST",
- dataType: "text",
- url: "/v1/logout",
- success: function() {
- window.location.replace("/");
- },
- failure: function() {
- window.location.replace("/");
- }
- });
+ var ua = window.navigator.userAgent;
+ if (ua.indexOf("MSIE ") > 0 || ua.indexOf(" Edge/") > 0 ||
ua.indexOf(" Trident/") > 0) {
+ $.ajax({
+ async: false,
--- End diff --
I'd avoid this at all costs. If the server is stalled or unreachable for a
short while the whole UI will freeze. See other comment for doing away with
this call altogether.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
