make a self-signed certificate on a non-primary domain a warning rather than an error, fixes #95

This commit is contained in:
Joshua Tauberer 2014-10-07 20:41:07 +00:00
parent 8566b78202
commit 0441a2e2e3
3 changed files with 18 additions and 6 deletions

View file

@ -255,6 +255,8 @@ def system_status():
self.items.append({ "type": "ok", "text": message, "extra": [] })
def print_error(self, message):
self.items.append({ "type": "error", "text": message, "extra": [] })
def print_warning(self, message):
self.items.append({ "type": "warning", "text": message, "extra": [] })
def print_line(self, message, monospace=False):
self.items[-1]["extra"].append({ "text": message, "monospace": monospace })
output = WebOutput()

View file

@ -386,7 +386,7 @@ def check_ssl_cert(domain, env):
env['out'].print_line("")
env['out'].print_line(" " + fingerprint, monospace=True)
else:
env['out'].print_error("""The SSL certificate for this domain is currently self-signed. Visitors to a website on
env['out'].print_warning("""The SSL certificate for this domain is currently self-signed. Visitors to a website on
this domain will get a security warning. If you are not serving a website on this domain, then it is
safe to leave the self-signed certificate in place.""")
env['out'].print_line("")
@ -571,6 +571,9 @@ class ConsoleOutput:
def print_error(self, message):
self.print_block(message, first_line="")
def print_warning(self, message):
self.print_block(message, first_line="? ")
def print_block(self, message, first_line=" "):
print(first_line, end='')
message = re.sub("\n\s*", " ", message)

View file

@ -10,11 +10,14 @@
border-top: none;
padding-top: 0;
}
#system-checks .error td {
#system-checks .status-error td {
color: #733;
}
#system-checks .ok td {
color: #030;
#system-checks .status-warning td {
color: #770;
}
#system-checks .status-ok td {
color: #040;
}
#system-checks div.extra {
display: none;
@ -52,9 +55,13 @@ function show_system_status() {
for (var i = 0; i < r.length; i++) {
var n = $("<tr><td class='status'/><td class='message'><p style='margin: 0'/><div class='extra'/><a class='showhide' href='#'/></tr>");
if (i == 0) n.addClass('first')
n.addClass(r[i].type)
if (r[i].type == "heading")
n.addClass(r[i].type)
else
n.addClass("status-" + r[i].type)
if (r[i].type == "ok") n.find('td.status').text("✓")
if (r[i].type == "error") n.find('td.status').text("✖")
if (r[i].type == "warning") n.find('td.status').text("?")
n.find('td.message p').text(r[i].text)
$('#system-checks tbody').append(n);