diff --git a/net/freeradius/Makefile b/net/freeradius/Makefile
index abfa82c67..3b4090d52 100644
--- a/net/freeradius/Makefile
+++ b/net/freeradius/Makefile
@@ -1,5 +1,6 @@
PLUGIN_NAME= freeradius
PLUGIN_VERSION= 1.9.28
+PLUGIN_REVISION= 1
PLUGIN_COMMENT= RADIUS Authentication, Authorization and Accounting Server
PLUGIN_DEPENDS= freeradius3
PLUGIN_MAINTAINER= m.muenz@gmail.com
diff --git a/net/freeradius/src/opnsense/mvc/app/models/OPNsense/Freeradius/General.xml b/net/freeradius/src/opnsense/mvc/app/models/OPNsense/Freeradius/General.xml
index 5e04541b4..754287d6e 100644
--- a/net/freeradius/src/opnsense/mvc/app/models/OPNsense/Freeradius/General.xml
+++ b/net/freeradius/src/opnsense/mvc/app/models/OPNsense/Freeradius/General.xml
@@ -25,7 +25,7 @@
4096
- You need to set a propper VLAN ID.
+ VLAN fallback assignment needs to be enabled.
DependConstraint
fallbackvlan_enabled
@@ -33,6 +33,18 @@
+
+ /^([0-9a-zA-Z._\-\!\$\%\/\(\)\+\#\=\{\}:]){1,128}$/u
+
+
+ VLAN fallback assignment needs to be enabled.
+ DependConstraint
+
+ fallbackvlan_enabled
+
+
+
+
0
N
@@ -152,8 +164,5 @@
0
Y
-
- /^([0-9a-zA-Z._\-\!\$\%\/\(\)\+\#\=\{\}:]){1,128}$/u
-
diff --git a/net/freeradius/src/opnsense/service/templates/OPNsense/Freeradius/users b/net/freeradius/src/opnsense/service/templates/OPNsense/Freeradius/users
index 9603b713c..4fa541ac7 100644
--- a/net/freeradius/src/opnsense/service/templates/OPNsense/Freeradius/users
+++ b/net/freeradius/src/opnsense/service/templates/OPNsense/Freeradius/users
@@ -113,14 +113,14 @@ DEFAULT Ldap-Group == "{{ ldapgroup_list.ldapgroupname }}"
{% endfor %}
{% endif %}
-{% if helpers.exists('OPNsense.freeradius.general.fallbackvlan_enabled') or helpers.exists('OPNsense.freeradius.general.fallback_tunnel_password') %}
+{% if OPNsense.freeradius.general.fallbackvlan_enabled == '1' %}
DEFAULT Auth-Type := Accept
-{% if helpers.exists('OPNsense.freeradius.general.fallbackvlan_enabled') and OPNsense.freeradius.general.fallbackvlan_enabled == '1' %}
Tunnel-Type = VLAN,
Tunnel-Medium-Type = IEEE-802,
+{% if helpers.exists('OPNsense.freeradius.general.fallbackvlan_id') and OPNsense.freeradius.general.fallbackvlan_id != '' %}
Tunnel-Private-Group-Id = {{ OPNsense.freeradius.general.fallbackvlan_id }},
{% endif %}
-{% if helpers.exists('OPNsense.freeradius.general.fallback_tunnel_password') %}
+{% if helpers.exists('OPNsense.freeradius.general.fallback_tunnel_password') and OPNsense.freeradius.general.fallback_tunnel_password != '' %}
Tunnel-Password = {{ OPNsense.freeradius.general.fallback_tunnel_password }},
{% endif %}
Framed-Protocol = PPP