base: minor logging updates, fixed port-argument in cli

Author: superstes

src/firewall_test/cli.py

@@ -81,6 +81,7 @@ def main():
             src=args.src_ip,
             dst=args.dst_ip,
             proto_l4=args.proto,
+            dport=args.port,
         )
 
     else:

src/firewall_test/simulator/firewall.py

@@ -62,7 +62,7 @@ def process(self, chain: Chain, packet: PACKET_KINDS) -> (bool, (Rule, None)):
                 log_info(
                     label='Firewall',
                     v1=f'> Chain {chain.name} | Rule {rule.seq}',
-                    v2=f' | {rule.dump()}'
+                    v2=f' | {rule.log()}'
                 )
                 continue
 
@@ -331,8 +331,8 @@ def _process_chain(self, chain: Chain, packet: PACKET_KINDS) -> (bool, (Rule, No
         log_info(
             'Firewall',
             f'Processing Chain: '
-            f'Table {chain.run_table.name} {chain.run_table.family.N} | '
-            f'Chain {chain.name} {chain.family.N} {chain.type}'
+            f'Table "{chain.run_table.name}" {chain.run_table.family.N} | '
+            f'Chain "{chain.name}" {chain.family.N} {chain.type} ({len(chain.rules)} rules)'
         )
         return self._run_chains.process(chain=chain, packet=packet)
 

src/firewall_test/utils/logger.py

@@ -123,7 +123,7 @@ def rule_repr(uid: (int, str), matches: any, cmt: str = None) -> str:
     if cmt is None:
         cmt = ''
 
-    elif not cmt.startswith(' '):
+    elif not cmt.startswith(' ') and cmt.strip() != '':
         cmt = f' "{cmt}"'
 
     if not isinstance(matches, str):