Skip to content

site: validate updated site configs

Warren Gifford requested to merge distribution/verify-updated-site-config into master

Created by: bobheadxi

closes #10883 (closed)

Adds a site config validation step before accepting input. The validation doesn't correctly check for licence key values (specifically mentioned in #10883 (closed)), so I've added a super simple check to error on the placeholder value. It's a little awkward - especially with a word like "license" which I often spell "licence", an exact string match is finicky. However, it doesn't look like there's a function for superficially validating a licence key outside of enterprise/internal, so I'm not sure if there's a better way to do this check 🤔 license key verification to conf.Validate

image

Merge request reports

Loading