relayd ion mode redirection gui blocked

[informatique4ipbs]

Important notices

Before you add a new report, we ask you kindly to acknowledge the following:

• I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
• I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug

Using relayd in relay mode for relaying my LDAP-Active Directory (port 389 and 3268) works like a charm. Everything is fine.
Using relayd in redirect mode fore relaying my DNS doesn t works. In fact when i try to implement it vi the gui all seems fine and when i click save it runs (like an infinite loop) and keeps the windows open. It never save the configuration (i ve search via a recursive grep in the conf ie : /etc , /usr/local/etc, /conf ). There is no error message.
i try to implement, at first the TCP part of example in manpage relayd :
redirect "dns" {
listen on dns.example.com tcp port 53
listen on dns.example.com udp port 53

               forward to <dnshosts> port 53 check tcp
       }

Expected behavior

create the virtual server

Describe alternatives you considered

????

Screenshots

2 screenshots one of relayd in relay mode (working) the other in redirect mode (not working)

Relevant log files

no as far as i 've ssen

Additional context

Add any other context about the problem here.

Environment

Software version used and hardware type if relevant, e.g.:

OPNsense 24.1.3_1-amd64
FreeBSD 13.2-RELEASE-p10
OpenSSL 3.0.13
everything is up to date

[mlevy823]

I have this same issue, the save button is nonfunctional if you select Redirection as a server type when you are on Edit Virtual Server. I am on a fresh and fully updated installation:
OPNsense 24.1.3_1-amd64
FreeBSD 13.2-RELEASE-p10
OpenSSL 3.0.13
os-relayd 2.8_1

[AdSchellevis]

when expanding advanced, there's likely an error. let me double check.

[AdSchellevis]

ed456b2 should fix the required but empty situation here.

[kulikov-a]

@AdSchellevis
Hi!
opnsense/core@9c98fd4
perhaps it wouldn’t be superfluous either?

[AdSchellevis]

@kulikov-a do you have an example form where this causes issues?

[kulikov-a]

@AdSchellevis
I tried it on the relayd Virtual Server form (without your model fix) and displaying a field validation error for the Backup Table Scheduler field (Scheduler "" not supported for redirects.) (index.volt view tries to keep track of how fields are displayed depending on the value of the options by assigning a .hidden class)

plugins/net/relayd/src/opnsense/mvc/app/views/OPNsense/Relayd/index.volt

Line 136 in cf4fa70

function ShowHideVSFields(){

[AdSchellevis]

@kulikov-a the relayd plugin seems to have more issues, I think we should refactor it one day to be honest. For now I only fixed the validation part.

[kulikov-a]

@AdSchellevis got it, thanks )