]> git.proxmox.com Git - proxmox-widget-toolkit.git/blobdiff - src/Utils.js
fix #4421: ui: guard setProxy against races of slow vs fast requests
[proxmox-widget-toolkit.git] / src / Utils.js
index d325644a429b3394ad99738b2977ad41f70aa04e..8a974870540b6d5e428267a2f59c27da62b04170 100644 (file)
@@ -306,7 +306,8 @@ utilities: {
        if (Proxmox.LoggedOut) {
            return;
        }
-       Ext.util.Cookies.clear(Proxmox.Setup.auth_cookie_name);
+       // ExtJS clear is basically the same, but browser may complain if any cookie isn't "secure"
+       Ext.util.Cookies.set(Proxmox.Setup.auth_cookie_name, "", new Date(0), null, null, true);
        window.localStorage.removeItem("ProxmoxUser");
     },
 
@@ -395,16 +396,15 @@ utilities: {
        if (!result.success) {
            msg = gettext("Unknown error");
            if (result.message) {
-               msg = result.message;
+               msg = Ext.htmlEncode(result.message);
                if (result.status) {
-                   msg += ' (' + result.status + ')';
+                   msg += ` (${result.status})`;
                }
            }
            if (verbose && Ext.isObject(result.errors)) {
                msg += "<br>";
-               Ext.Object.each(result.errors, function(prop, desc) {
-                   msg += "<br><b>" + Ext.htmlEncode(prop) + "</b>: " +
-                       Ext.htmlEncode(desc);
+               Ext.Object.each(result.errors, (prop, desc) => {
+                   msg += `<br><b>${Ext.htmlEncode(prop)}</b>: ${Ext.htmlEncode(desc)}`;
                });
            }
        }
@@ -418,6 +418,10 @@ utilities: {
            waitMsg: gettext('Please wait...'),
        }, reqOpts);
 
+       // default to enable if user isn't handling the failure already explicitly
+       let autoErrorAlert = reqOpts.autoErrorAlert ??
+           (typeof reqOpts.failure !== 'function' && typeof reqOpts.callback !== 'function');
+
        if (!newopts.url.match(/^\/api2/)) {
            newopts.url = '/api2/extjs' + newopts.url;
        }
@@ -439,6 +443,9 @@ utilities: {
                        response.htmlStatus = Proxmox.Utils.extractRequestError(result, true);
                        Ext.callback(callbackFn, options.scope, [options, false, response]);
                        Ext.callback(failureFn, options.scope, [response, options]);
+                       if (autoErrorAlert) {
+                           Ext.Msg.alert(gettext('Error'), response.htmlStatus);
+                       }
                        return;
                    }
                    Ext.callback(callbackFn, options.scope, [options, true, response]);
@@ -568,7 +575,7 @@ utilities: {
            return;
        }
 
-       let items = container.query('>'); // direct childs
+       let items = container.query('>'); // direct children
        factor = Math.min(factor, items.length);
        container.oldFactor = factor;
 
@@ -1271,6 +1278,107 @@ utilities: {
            .map(val => val.charCodeAt(0)),
        );
     },
+
+    stringToRGB: function(string) {
+       let hash = 0;
+       if (!string) {
+           return hash;
+       }
+       string += 'prox'; // give short strings more variance
+       for (let i = 0; i < string.length; i++) {
+           hash = string.charCodeAt(i) + ((hash << 5) - hash);
+           hash = hash & hash; // to int
+       }
+
+       let alpha = 0.7; // make the color a bit brighter
+       let bg = 255; // assume white background
+
+       return [
+           (hash & 255) * alpha + bg * (1 - alpha),
+           ((hash >> 8) & 255) * alpha + bg * (1 - alpha),
+           ((hash >> 16) & 255) * alpha + bg * (1 - alpha),
+       ];
+    },
+
+    rgbToCss: function(rgb) {
+       return `rgb(${rgb[0]}, ${rgb[1]}, ${rgb[2]})`;
+    },
+
+    rgbToHex: function(rgb) {
+       let r = Math.round(rgb[0]).toString(16);
+       let g = Math.round(rgb[1]).toString(16);
+       let b = Math.round(rgb[2]).toString(16);
+       return `${r}${g}${b}`;
+    },
+
+    hexToRGB: function(hex) {
+       if (!hex) {
+           return undefined;
+       }
+       if (hex.length === 7) {
+           hex = hex.slice(1);
+       }
+       let r = parseInt(hex.slice(0, 2), 16);
+       let g = parseInt(hex.slice(2, 4), 16);
+       let b = parseInt(hex.slice(4, 6), 16);
+       return [r, g, b];
+    },
+
+    // optimized & simplified SAPC function
+    // https://github.com/Myndex/SAPC-APCA
+    getTextContrastClass: function(rgb) {
+           const blkThrs = 0.022;
+           const blkClmp = 1.414;
+
+           // linearize & gamma correction
+           let r = (rgb[0] / 255) ** 2.4;
+           let g = (rgb[1] / 255) ** 2.4;
+           let b = (rgb[2] / 255) ** 2.4;
+
+           // relative luminance sRGB
+           let bg = r * 0.2126729 + g * 0.7151522 + b * 0.0721750;
+
+           // black clamp
+           bg = bg > blkThrs ? bg : bg + (blkThrs - bg) ** blkClmp;
+
+           // SAPC with white text
+           let contrastLight = bg ** 0.65 - 1;
+           // SAPC with black text
+           let contrastDark = bg ** 0.56 - 0.046134502;
+
+           if (Math.abs(contrastLight) >= Math.abs(contrastDark)) {
+               return 'light';
+           } else {
+               return 'dark';
+           }
+    },
+
+    getTagElement: function(string, color_overrides) {
+       let rgb = color_overrides?.[string] || Proxmox.Utils.stringToRGB(string);
+       let style = `background-color: ${Proxmox.Utils.rgbToCss(rgb)};`;
+       let cls;
+       if (rgb.length > 3) {
+           style += `color: ${Proxmox.Utils.rgbToCss([rgb[3], rgb[4], rgb[5]])}`;
+           cls = "proxmox-tag-dark";
+       } else {
+           let txtCls = Proxmox.Utils.getTextContrastClass(rgb);
+           cls = `proxmox-tag-${txtCls}`;
+       }
+       return `<span class="${cls}" style="${style}">${string}</span>`;
+    },
+
+    // Setting filename here when downloading from a remote url sometimes fails in chromium browsers
+    // because of a bug when using attribute download in conjunction with a self signed certificate.
+    // For more info see https://bugs.chromium.org/p/chromium/issues/detail?id=993362
+    downloadAsFile: function(source, fileName) {
+       let hiddenElement = document.createElement('a');
+       hiddenElement.href = source;
+       hiddenElement.target = '_blank';
+       if (fileName) {
+           hiddenElement.download = fileName;
+       }
+       hiddenElement.click();
+    },
 },
 
     singleton: true,
@@ -1314,6 +1422,8 @@ utilities: {
        me.DnsName_match = new RegExp("^" + DnsName_REGEXP + "$");
        me.DnsName_or_Wildcard_match = new RegExp("^(?:\\*\\.)?" + DnsName_REGEXP + "$");
 
+       me.CpuSet_match = /^[0-9]+(?:-[0-9]+)?(?:,[0-9]+(?:-[0-9]+)?)*$/;
+
        me.HostPort_match = new RegExp("^(" + IPV4_REGEXP + "|" + DnsName_REGEXP + ")(?::(\\d+))?$");
        me.HostPortBrackets_match = new RegExp("^\\[(" + IPV6_REGEXP + "|" + IPV4_REGEXP + "|" + DnsName_REGEXP + ")\\](?::(\\d+))?$");
        me.IP6_dotnotation_match = new RegExp("^(" + IPV6_REGEXP + ")(?:\\.(\\d+))?$");
@@ -1326,7 +1436,7 @@ Ext.define('Proxmox.Async', {
     singleton: true,
 
     // Returns a Promise resolving to the result of an `API2Request` or rejecting to the error
-    // repsonse on failure
+    // response on failure
     api2: function(reqOpts) {
        return new Promise((resolve, reject) => {
            delete reqOpts.callback; // not allowed in this api
@@ -1341,3 +1451,18 @@ Ext.define('Proxmox.Async', {
        return new Promise((resolve, _reject) => setTimeout(resolve, millis));
     },
 });
+
+Ext.override(Ext.data.Store, {
+    // If the store's proxy is changed while it is waiting for an AJAX
+    // response, `onProxyLoad` will still be called for the outdated response.
+    // To avoid displaying inconsistent information, only process responses
+    // belonging to the current proxy.
+    onProxyLoad: function(operation) {
+       let me = this;
+       if (operation.getProxy() === me.getProxy()) {
+           me.callParent(arguments);
+       } else {
+           console.log(`ignored outdated response: ${operation.getRequest().getUrl()}`);
+       }
+    },
+});