Skip to content

Defguard Gateway service not registered on opnsense in 1.6.3#278

Merged
moubctez merged 1 commit intoDefGuard:mainfrom
jamtur01:enabled
Feb 23, 2026
Merged

Defguard Gateway service not registered on opnsense in 1.6.3#278
moubctez merged 1 commit intoDefGuard:mainfrom
jamtur01:enabled

Conversation

@jamtur01
Copy link
Contributor

📖 Description

In 1.6.3 the Defguard Gateway service is not being registered. This is because of a mismatch between the field name and case. OPNsense's MVC framework stores XML model field names exactly as defined. The model defines with a capital E, and that's how it appears in the $config array at runtime.

In our config, half the cases look for 'enabled' and the other half look for 'Enabled'. Making it consistent.

🛠️ Dev Branch Merge Checklist:

Documentation

  • [NA] If testing requires changes in the environment or deployment, please update the documentation (https://defguard.gitbook.io) first and attach the link to the documentation section in this pool request
  • [NA] I have commented on my code, particularly in hard-to-understand areas

Testing

  • [NA] I have prepared end-to-end tests for all new functionalities
  • I have performed end-to-end tests manually and they work
  • New and existing unit tests pass locally with my changes

Deployment

  • [NA] If deployment is affected I have made corresponding/required changes to deployment (Docker, Kubernetes, one-line install)

🏚️ Main Branch Merge Checklist:

Testing

  • [NA] I have merged my changes before to dev and the dev checklist is done
  • [NA] I have tested all functionalities on the dev instance and they work

Documentation

  • [NA] I have made corresponding changes to the user & admin documentation and added new features documentation with screenshots for users/admins

…s is because of a mismatch between field name case. OPNsense's MVC framework stores XML model field names exactly as defined. The model defines <Enabled> with a capital E, and that's how it appears in the $config array at runtime.

In our config we have half the cases looking for 'enabled' and the other
looking for 'Enabled'. Making it consistent.
@teon teon requested review from moubctez and wojcik91 February 21, 2026 12:15
@moubctez moubctez removed the request for review from wojcik91 February 23, 2026 08:16
@moubctez moubctez merged commit affd9b4 into DefGuard:main Feb 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants