From c5b2e6d9452296522835d07ea53eca42dc9355e3 Mon Sep 17 00:00:00 2001
From: Rhonda D'Vine <rhonda@proxmox.com>
Date: Mon, 12 Nov 2018 15:14:45 +0100
Subject: [PATCH] Fix #1971: display firewall rule properties

This is the list of the properties that should get returned in the
pretty print format, too, not just in yaml/json output.

Signed-off-by: Rhonda D'Vine <rhonda@proxmox.com>
---
 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..b2a81a1 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 => 'integer',
+		    optional => 1,
+		},
+		iface => {
+		    type => 'string',
+		    optional => 1,
+		},
+		ipversion => {
+		    type => 'integer',
+		    optional => 1,
+		},
+		macro => {
+		    type => 'integer',
+		    optional => 1,
+		},
 		pos => {
 		    type => 'integer',
-		}
+		},
+		proto => {
+		    type => 'string',
+		    optional => 1,
+		},
+		source => {
+		    type => 'string',
+		    optional => 1,
+		},
+		sport => {
+		    type => 'string',
+		    optional => 1,
+		},
+		type => {
+		    type => 'string',
+		},
 	    },
 	},
 	code => sub {
-- 
2.39.2