From c574607173b4db3608f44d73b3c4fdb77113b28a Mon Sep 17 00:00:00 2001
From: Eduardo Silva <eduardo@eth0.com.br>
Date: Thu, 13 Mar 2025 09:46:39 -0300
Subject: [PATCH] include destination port information when used

---
 templates/firewall/redirect_rule_list.html | 50 ++++++++--------------
 1 file changed, 17 insertions(+), 33 deletions(-)

diff --git a/templates/firewall/redirect_rule_list.html b/templates/firewall/redirect_rule_list.html
index 95291b3..d361e39 100644
--- a/templates/firewall/redirect_rule_list.html
+++ b/templates/firewall/redirect_rule_list.html
@@ -1,14 +1,13 @@
 {% extends 'base.html' %}
 
 {% block content %}
-
-<div class="card card-primary card-outline">
-    <div class="card-body">
-        {% include "firewall/firewall_nav_tabs.html" %}
-        <div class="tab-content" id="custom-content-below-tabContent">
-            <div class="tab-pane fade show active" id="custom-content-below-home" role="tabpanel" aria-labelledby="custom-content-below-home-tab">
-                <table class="table table-striped table-hover">
-                    <thead>
+    <div class="card card-primary card-outline">
+        <div class="card-body">
+            {% include "firewall/firewall_nav_tabs.html" %}
+            <div class="tab-content" id="custom-content-below-tabContent">
+                <div class="tab-pane fade show active" id="custom-content-below-home" role="tabpanel" aria-labelledby="custom-content-below-home-tab">
+                    <table class="table table-striped table-hover">
+                        <thead>
                         <tr>
                             <th>Instance</th>
                             <th>Protocol</th>
@@ -18,8 +17,8 @@
                             <th>Masquerade Source</th>
                             <th><i class="far fa-edit"></i></th>
                         </tr>
-                    </thead>
-                    <tbody>
+                        </thead>
+                        <tbody>
                         {% for redirect_rule in redirect_rule_list %}
                             <tr>
                                 <td>{{ redirect_rule.wireguard_instance }}</td>
@@ -28,25 +27,18 @@
                                 <td>
                                     {% if redirect_rule.peer %}
                                         <a href="/peer/manage/?peer={{ redirect_rule.peer.uuid }}">
-                                        {% if redirect_rule.peer.name %}
-                                            {{ redirect_rule.peer.name }}
-                                        {% else %}
-                                            {{ redirect_rule.peer.public_key|slice:":16" }}{% if redirect_rule.peer.public_key|length > 16 %}...{% endif %}
-                                        {% endif %}
-                                        </a>                    
+                                            {{ redirect_rule.peer }}{% if redirect_rule.port_forward %}:{{ redirect_rule.port_forward }}{% endif %}
+                                        </a>
                                     {% else %}
-                                        {{ redirect_rule.ip_address }}
+                                        {{ redirect_rule.ip_address }}{% if redirect_rule.port_forward %}:{{ redirect_rule.port_forward }}{% endif %}
                                     {% endif %}
                                 </td>
                                 <td>
-                                    
                                     {% if redirect_rule.add_forward_rule %}
                                         <i class="fas fa-check"></i>
                                     {% else %}
                                         <i class="fas fa-times"></i>
-                
                                     {% endif %}
-                                
                                 </td>
                                 <td>
                                     {% if redirect_rule.masquerade_source %}
@@ -54,27 +46,19 @@
                                         <i class="fas fa-exclamation-triangle" title="This serves as a temporary solution when a peer does not use the VPN as its default gateway. It's important to note that this configuration is not recommended, as it alters the source address of all connections to match the IP address of the WireGuard instance."></i>
                                     {% else %}
                                         <i class="fas fa-times"></i>
-                
                                     {% endif %}
-                
                                 </td>
                                 <td style="width: 1%; white-space: nowrap;">
                                     <a href="/firewall/manage_port_forward_rule/?uuid={{ redirect_rule.uuid }}" ><i class="far fa-edit"></i></a>
                                 </td>
                             </tr>
                         {% endfor %}
-                
-                    </tbody>
-                </table>
-                <a href="/firewall/manage_port_forward_rule/" class='btn btn-primary'>Create Port forwarding Rule</a>
-                <a href="/firewall/firewall_settings/?chain=porforward" class='btn btn-outline-primary'>Firewall Settings</a>
+                        </tbody>
+                    </table>
+                    <a href="/firewall/manage_port_forward_rule/" class='btn btn-primary'>Create Port forwarding Rule</a>
+                    <a href="/firewall/firewall_settings/?chain=porforward" class='btn btn-outline-primary'>Firewall Settings</a>
+                </div>
             </div>
         </div>
     </div>
-</div>
-
-
-
-
-
 {% endblock %}
\ No newline at end of file