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

configmgr schema validation regression on null, required fields #458

Open
1000TurquoisePogs opened this issue Jun 21, 2024 · 0 comments
Open
Labels
bug Something isn't working new not yet triaged

Comments

@1000TurquoisePogs
Copy link
Member

1000TurquoisePogs commented Jun 21, 2024

v2.16.0's bugfix of yaml nulls being treated as strings #435 has caused user bugs from configs prior to v2.9.0 (June 2023),

Before that time, users of keyrings would have in their configs

pem:
  key:
  certificate:
  certificateAuthority:
  - A
  - B

key & certificate are listed as "required" fields. So, though configmgr is now correctly identifying this as invalid (a required field shouldnt be null), users depended upon this and thus we should restore this behavior until for example v3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working new not yet triaged
Projects
None yet
Development

No branches or pull requests

1 participant