Validate value of domain property

[ppawlowski]

Description

This PR assures, that forge.domain value contains a valid domain name.
In result, installation without proper domain set will fail with:

Error: UPGRADE FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
flowforge:
- forge.domain: Does not match pattern '^([a-z0-9]+(-[a-z0-9]+)*\.)+[a-z]{2,6}$'

Related Issue(s)

#226

Checklist

• I have read the contribution guidelines
• Suitable unit/system level tests have been added and they pass
• Documentation has been updated
  • Upgrade instructions
  • Configuration details
  • Concepts
• Changes flowforge.yml?
  • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
  • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production

Labels

• Backport needed? -> add the backport label
• Includes a DB migration? -> add the area:migration label

[hardillb]

is the \\ because we need to escape the escape char in the sting?

[hardillb]

I see you've limited top level domains to between 2-6 chars

This is the current list which already has longer version (e.g. 11 for ACCOUNTANTS, or 15 for AMERICANEXPRESS )
https://data.iana.org/TLD/tlds-alpha-by-domain.txt

https://developer.mozilla.org/en-US/docs/Learn/Common_questions/Web_mechanics/What_is_a_domain_name

TLDs can contain special as well as latin characters. A TLD's maximum length is 63 characters, although most are around 2–3.

[ppawlowski]

is the \ because we need to escape the escape char in the sting?
Yes

[ppawlowski]

Applied a change which allows TLD to be 25 characters - the longest from the IANA list at the time of writing.

[hardillb]

This would mean people can't run forge on a top level domain, but I'm happy with that for now...