[pve-devel] [PATCH firewall 1/2] Fix #1971: display firewall rule properties
Rhonda D'Vine
rhonda at proxmox.com
Mon Nov 12 15:10:18 CET 2018
Hi.
On 11/12/18 12:49 PM, Dominik Csapak wrote:
> On 11/12/18 12:27 PM, Rhonda D'Vine wrote:
>> ---
>> src/PVE/API2/Firewall/Rules.pm | 48
>> +++++++++++++++++++++++++++++++++++++++++-
>> 1 file changed, 47 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/PVE/API2/Firewall/Rules.pm
>> b/src/PVE/API2/Firewall/Rules.pm
>> index 1aa6d27..52a0935 100644
>> --- a/src/PVE/API2/Firewall/Rules.pm
>> +++ b/src/PVE/API2/Firewall/Rules.pm
>> @@ -122,9 +122,55 @@ sub register_get_rule {
>> returns => {
>> type => "object",
>> properties => {
>> + action => {
>> + type => 'string',
>> + },
>> + comment => {
>> + type => 'string',
>> + optional => 1,
>> + },
>> + dest => {
>> + type => 'string',
>> + optional => 1,
>> + },
>> + dport => {
>> + type => 'string',
>> + optional => 1,
>> + },
>> + enable => {
>> + type => 'intager',
>
> should probably be 'integer' ?
Good catch - no clue how that slipped in. And what confuses me even
more that it doesn't seem to make a difference in the code because
that's what I have on my system and tested with. *puzzled*
v2 will come shortly.
Rhonda
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://lists.proxmox.com/pipermail/pve-devel/attachments/20181112/3c16116c/attachment.sig>
More information about the pve-devel
mailing list