fix #2632: use alternative api call for user black/whitelist

which expects the entry as parameter instead of part of the path
this allows all special characters to be included in the entry

in the url, some characters were problematic (e.g. '/') and not properly
handled by the api in the backend

we use Ext.applyIf (which copies the non-existant entries to params) here
instead of getting the ExtraParams from the proxy and setting '.address'
(that would modify the ExtraParams of the proxy and send unwanted
parameters to subsequent api calls like the list-loading)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Dominik Csapak 2020-03-11 12:03:34 +01:00 committed by Thomas Lamprecht
parent 4460d9531d
commit a144e5343d

View File

@ -73,11 +73,10 @@ Ext.define('PMG.UserBlackWhiteList', {
var url = '/quarantine/' + me.listname + '/';
url += records.map(function(rec) {
return encodeURIComponent(rec.getId());
}).join(',');
var params = me.getStore().getProxy().getExtraParams() || {};
let params = {
address: records.map((rec) => rec.getId()).join(','),
};
Ext.applyIf(params, me.getStore().getProxy().getExtraParams());
Proxmox.Utils.API2Request({
url: url + '?' + Ext.Object.toQueryString(params),