diff --git a/webadmin/admin.go b/webadmin/admin.go index 18fb747eaf..bd2f46ea6a 100644 --- a/webadmin/admin.go +++ b/webadmin/admin.go @@ -1404,7 +1404,11 @@ When enabling MTA-STS, or updating a policy, always update the policy first (thr if err != nil { addf(&r.SRVConf.Errors, "Looking up SRV record %q: %s", name, err) } else if len(req.srvs) == 0 { - addf(&r.SRVConf.Errors, "Missing SRV record %q", name) + if req.host == "." { + addf(&r.SRVConf.Warnings, "Missing optional SRV record %q", name) + } else { + addf(&r.SRVConf.Errors, "Missing SRV record %q", name) + } } else if len(req.srvs) != 1 || req.srvs[0].Target != req.host || req.srvs[0].Port != req.port { addf(&r.SRVConf.Errors, "Unexpected SRV record(s) for %q", name) } diff --git a/webadmin/admin.js b/webadmin/admin.js index 36bf6c221a..3ec81366f5 100644 --- a/webadmin/admin.js +++ b/webadmin/admin.js @@ -2592,8 +2592,8 @@ const domainDNSCheck = async (d) => { if ((r.Errors || []).length === 0 && (r.Warnings || []).length === 0) { success = box(green, 'OK'); } - const errors = (r.Errors || []).length === 0 ? [] : box(red, dom.ul(style({ marginLeft: '1em' }), (r.Errors || []).map(s => dom.li(s)))); - const warnings = (r.Warnings || []).length === 0 ? [] : box(yellow, dom.ul(style({ marginLeft: '1em' }), (r.Warnings || []).map(s => dom.li(s)))); + const errors = (r.Errors || []).length === 0 ? [] : box(red, dom.ul((r.Errors || []).map(s => dom.li(s)))); + const warnings = (r.Warnings || []).length === 0 ? [] : box(yellow, dom.ul((r.Warnings || []).map(s => dom.li(s)))); let instructions = null; if (r.Instructions && r.Instructions.length > 0) { instructions = dom.div(style({ margin: '.5ex 0' })); diff --git a/webadmin/admin.ts b/webadmin/admin.ts index de6e70a517..393939df15 100644 --- a/webadmin/admin.ts +++ b/webadmin/admin.ts @@ -1928,8 +1928,8 @@ const domainDNSCheck = async (d: string) => { if ((r.Errors || []).length === 0 && (r.Warnings || []).length === 0) { success = box(green, 'OK') } - const errors = (r.Errors || []).length === 0 ? [] : box(red, dom.ul(style({marginLeft: '1em'}), (r.Errors || []).map(s => dom.li(s)))) - const warnings = (r.Warnings || []).length === 0 ? [] : box(yellow, dom.ul(style({marginLeft: '1em'}), (r.Warnings || []).map(s => dom.li(s)))) + const errors = (r.Errors || []).length === 0 ? [] : box(red, dom.ul((r.Errors || []).map(s => dom.li(s)))) + const warnings = (r.Warnings || []).length === 0 ? [] : box(yellow, dom.ul((r.Warnings || []).map(s => dom.li(s)))) let instructions: HTMLElement | null = null if (r.Instructions && r.Instructions.length > 0) {