Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature] strict config validation #1926

Open
2 tasks
SuperSandro2000 opened this issue May 5, 2024 · 1 comment
Open
2 tasks

[Feature] strict config validation #1926

SuperSandro2000 opened this issue May 5, 2024 · 1 comment
Labels
enhancement New feature or request needs design doc

Comments

@SuperSandro2000
Copy link
Contributor

SuperSandro2000 commented May 5, 2024

Use case

I've run multiple times into issues because I've miss typed a config key name or forgot to copy a letter. Headscale should catch that and fail early in when loading the config and complain about keys it doesn't know.

Description

I am imagining something like json's DisallowUnknownFields() which complains if any field cannot be mapped into a struct.

Contribution

  • I can write the design doc for this feature
  • I can contribute this feature

How can it be implemented?

No response

@SuperSandro2000 SuperSandro2000 added the enhancement New feature or request label May 5, 2024
@kradalby
Copy link
Collaborator

kradalby commented May 8, 2024

I agree, the whole config management is a bit of a pain, I particularly does not like that it can only be loaded and not easily dumped at the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs design doc
Projects
None yet
Development

No branches or pull requests

2 participants