opnsense / plugins

OPNsense plugin collection
https://opnsense.org/
BSD 2-Clause "Simplified" License
810 stars 593 forks source link

Haproxy, Using ${var} in strings is deprecated, use {$var} instead #3991

Closed zaidranger closed 1 month ago

zaidranger commented 1 month ago

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

Describe the bug Cannot configure haproxy even after closing the danger popout warnings.

To Reproduce Steps to reproduce the behavior:

  1. Go to 'haproxy'
  2. Click on 'anything from the sidemenu'
  3. See error

Expected behavior no dangers popout or can configure haproxy settings after closing said popouts

Additional context I am aware that these will be replaced but as of now there is no way of configuring haproxy as long as the error is there, manually changing to {$var} fixes the issue but it's there again after an update.

zaidranger commented 1 month ago

Manually replaced the $(var) with ($var) and it worked again....hope it survives an update.

fichtner commented 1 month ago

It will not. Happy if you could provide a PR.

zaidranger commented 1 month ago

4005

This fix is also applicable for acmeclient

fichtner commented 1 month ago

Thanks a lot! I forgot to mention that you probably have development mode enabled which is the only way this could break the API here?

zaidranger commented 1 month ago

Yeah i'm in development, i think that might be the case.