net/frr: Add checkbox for bgp neighbor changes (#3490)

net/frr/Makefile

@@ -1,5 +1,5 @@
 PLUGIN_NAME=		frr
-PLUGIN_VERSION=		1.33
+PLUGIN_VERSION=		1.34
 PLUGIN_COMMENT=		The FRRouting Protocol Suite
 PLUGIN_DEPENDS=		frr8
 PLUGIN_MAINTAINER=	franz.fabian.94@gmail.com

net/frr/pkg-descr

@@ -12,6 +12,10 @@ WWW: https://frrouting.org/
 Plugin Changelog
 ================
 
+1.34
+
+* Add checkbox to log BGP neighbor changes
+
 1.33
 
 * Allow both IPv4 and IPv6 in BGP via setkey (contributed by as8net)

net/frr/src/opnsense/mvc/app/controllers/OPNsense/Quagga/forms/bgp.xml

@@ -40,6 +40,12 @@
         <advanced>true</advanced>
         <help>When enabled (default), BGP only announces networks set at 'Network' if they are present in the routers routing table (alternatively, you can also set a null-route via System -> Routes). If disabled, all configured networks will be announced.</help>
     </field>
+    <field>
+        <id>bgp.logneighborchanges</id>
+        <label>Log Neighbor Changes</label>
+        <type>checkbox</type>
+        <help>This will activate exetended logging of BGP neighbor changes.</help>
+    </field>
     <field>
         <id>bgp.redistribute</id>
         <label>Route Redistribution</label>

net/frr/src/opnsense/mvc/app/models/OPNsense/Quagga/BGP.xml

@@ -25,6 +25,10 @@
             <default>1</default>
             <Required>Y</Required>
         </networkimportcheck>
+        <logneighborchanges type="BooleanField">
+            <default>0</default>
+            <Required>Y</Required>
+        </logneighborchanges>
         <networks type="CSVListField">
             <default></default>
             <Required>N</Required>

net/frr/src/opnsense/service/templates/OPNsense/Quagga/bgpd.conf

@@ -43,6 +43,9 @@ frr defaults {{ OPNsense.quagga.general.profile }}
 !
 {% if helpers.exists('OPNsense.quagga.bgp.asnumber') and OPNsense.quagga.bgp.asnumber != '' %}
 router bgp {{ OPNsense.quagga.bgp.asnumber }}
+{%   if not helpers.empty('OPNsense.quagga.bgp.logneighborchanges') %}
+ bgp log-neighbor-changes
+{%   endif %}
  no bgp default ipv4-unicast
  no bgp ebgp-requires-policy
 {%   if helpers.exists('OPNsense.quagga.bgp.networkimportcheck') and OPNsense.quagga.bgp.networkimportcheck == '1' %}