[pve-devel] [PATCH manager 1/2] add htmlEncode to various fields
Dominik Csapak
d.csapak at proxmox.com
Thu Feb 18 15:21:39 CET 2016
add htmlEnode as a renderer to fields which lets you
input arbitrary strings, to prevent html tags
Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
www/manager/dc/AuthView.js | 1 +
www/manager/dc/GroupView.js | 1 +
www/manager/dc/PoolView.js | 1 +
www/manager/dc/SecurityGroups.js | 2 +-
www/manager/dc/UserView.js | 1 +
www/manager/grid/FirewallAliases.js | 2 +-
www/manager/ha/Groups.js | 3 ++-
www/manager/ha/Resources.js | 3 ++-
www/manager/lxc/SnapshotTree.js | 2 +-
www/manager/panel/IPSet.js | 2 +-
www/manager/pool/StatusView.js | 1 +
www/manager/qemu/Options.js | 1 +
www/manager/qemu/SnapshotTree.js | 2 +-
13 files changed, 15 insertions(+), 7 deletions(-)
diff --git a/www/manager/dc/AuthView.js b/www/manager/dc/AuthView.js
index 83e79c6..2a2961c 100644
--- a/www/manager/dc/AuthView.js
+++ b/www/manager/dc/AuthView.js
@@ -132,6 +132,7 @@ Ext.define('PVE.dc.AuthView', {
header: gettext('Comment'),
sortable: false,
dataIndex: 'comment',
+ renderer: Ext.String.htmlEncode,
flex: 1
}
],
diff --git a/www/manager/dc/GroupView.js b/www/manager/dc/GroupView.js
index 6950a46..9612721 100644
--- a/www/manager/dc/GroupView.js
+++ b/www/manager/dc/GroupView.js
@@ -95,6 +95,7 @@ Ext.define('PVE.dc.GroupView', {
{
header: gettext('Comment'),
sortable: false,
+ renderer: Ext.String.htmlEncode,
dataIndex: 'comment',
flex: 1
}
diff --git a/www/manager/dc/PoolView.js b/www/manager/dc/PoolView.js
index 4ae99e2..0d552ec 100644
--- a/www/manager/dc/PoolView.js
+++ b/www/manager/dc/PoolView.js
@@ -96,6 +96,7 @@ Ext.define('PVE.dc.PoolView', {
header: gettext('Comment'),
sortable: false,
dataIndex: 'comment',
+ renderer: Ext.String.htmlEncode,
flex: 1
}
],
diff --git a/www/manager/dc/SecurityGroups.js b/www/manager/dc/SecurityGroups.js
index 0e31295..d8562e7 100644
--- a/www/manager/dc/SecurityGroups.js
+++ b/www/manager/dc/SecurityGroups.js
@@ -167,7 +167,7 @@ Ext.define('PVE.SecurityGroupList', {
selModel: sm,
columns: [
{ header: gettext('Group'), dataIndex: 'group', width: 100 },
- { header: gettext('Comment'), dataIndex: 'comment', flex: 1 }
+ { header: gettext('Comment'), dataIndex: 'comment', renderer: Ext.String.htmlEncode, flex: 1 }
],
listeners: {
itemdblclick: run_editor,
diff --git a/www/manager/dc/UserView.js b/www/manager/dc/UserView.js
index c4f8a8b..d4ad713 100644
--- a/www/manager/dc/UserView.js
+++ b/www/manager/dc/UserView.js
@@ -220,6 +220,7 @@ Ext.define('PVE.dc.UserView', {
id: 'comment',
header: gettext('Comment'),
sortable: false,
+ renderer: Ext.String.htmlEncode,
dataIndex: 'comment',
flex: 1
}
diff --git a/www/manager/grid/FirewallAliases.js b/www/manager/grid/FirewallAliases.js
index 353b97c..2f38393 100644
--- a/www/manager/grid/FirewallAliases.js
+++ b/www/manager/grid/FirewallAliases.js
@@ -171,7 +171,7 @@ Ext.define('PVE.FirewallAliases', {
columns: [
{ header: gettext('Name'), dataIndex: 'name', width: 100 },
{ header: gettext('IP/CIDR'), dataIndex: 'cidr', width: 100 },
- { header: gettext('Comment'), dataIndex: 'comment', flex: 1 }
+ { header: gettext('Comment'), dataIndex: 'comment', renderer: Ext.String.htmlEncode, flex: 1 }
],
listeners: {
itemdblclick: run_editor
diff --git a/www/manager/ha/Groups.js b/www/manager/ha/Groups.js
index e596c00..28be5f7 100644
--- a/www/manager/ha/Groups.js
+++ b/www/manager/ha/Groups.js
@@ -108,7 +108,8 @@ Ext.define('PVE.ha.GroupsView', {
{
header: gettext('Comment'),
flex: 1,
- dataIndex: 'comment'
+ dataIndex: 'comment',
+ renderer: Ext.String.htmlEncode
}
],
listeners: {
diff --git a/www/manager/ha/Resources.js b/www/manager/ha/Resources.js
index 495f039..87a9723 100644
--- a/www/manager/ha/Resources.js
+++ b/www/manager/ha/Resources.js
@@ -133,7 +133,8 @@ Ext.define('PVE.ha.ResourcesView', {
{
header: gettext('Description'),
flex: 1,
- dataIndex: 'comment'
+ dataIndex: 'comment',
+ renderer: Ext.String.htmlEncode
}
],
listeners: {
diff --git a/www/manager/lxc/SnapshotTree.js b/www/manager/lxc/SnapshotTree.js
index f13e64f..cbe7043 100644
--- a/www/manager/lxc/SnapshotTree.js
+++ b/www/manager/lxc/SnapshotTree.js
@@ -279,7 +279,7 @@ Ext.define('PVE.lxc.SnapshotTree', {
if (record.data.name === 'current') {
return gettext("You are here!");
} else {
- return value;
+ return Ext.String.htmlEncode(value);
}
}
}
diff --git a/www/manager/panel/IPSet.js b/www/manager/panel/IPSet.js
index 2aae7df..821f467 100644
--- a/www/manager/panel/IPSet.js
+++ b/www/manager/panel/IPSet.js
@@ -148,7 +148,7 @@ Ext.define('PVE.IPSetList', {
selModel: sm,
columns: [
{ header: 'IPSet', dataIndex: 'name', width: 100 },
- { header: gettext('Comment'), dataIndex: 'comment', flex: 1 }
+ { header: gettext('Comment'), dataIndex: 'comment', renderer: Ext.String.htmlEncode, flex: 1 }
],
listeners: {
itemdblclick: run_editor,
diff --git a/www/manager/pool/StatusView.js b/www/manager/pool/StatusView.js
index 8049364..a376e09 100644
--- a/www/manager/pool/StatusView.js
+++ b/www/manager/pool/StatusView.js
@@ -13,6 +13,7 @@ Ext.define('PVE.pool.StatusView', {
var rows = {
comment: {
header: gettext('Comment'),
+ renderer: Ext.String.htmlEncode,
required: true
}
};
diff --git a/www/manager/qemu/Options.js b/www/manager/qemu/Options.js
index c31af13..9ba913c 100644
--- a/www/manager/qemu/Options.js
+++ b/www/manager/qemu/Options.js
@@ -242,6 +242,7 @@ Ext.define('PVE.qemu.Options', {
smbios1: {
header: gettext('SMBIOS settings (type1)'),
defaultValue: '',
+ renderer: Ext.String.htmlEncode,
editor: caps.vms['VM.Config.HWType'] ? 'PVE.qemu.Smbios1Edit' : undefined
},
agent: {
diff --git a/www/manager/qemu/SnapshotTree.js b/www/manager/qemu/SnapshotTree.js
index 6f377a2..6efa944 100644
--- a/www/manager/qemu/SnapshotTree.js
+++ b/www/manager/qemu/SnapshotTree.js
@@ -279,7 +279,7 @@ Ext.define('PVE.qemu.SnapshotTree', {
if (record.data.name === 'current') {
return gettext("You are here!");
} else {
- return value;
+ return Ext.String.htmlEncode(value);
}
}
}
--
2.1.4
More information about the pve-devel
mailing list