From c5c670344014aa5f6226ee5470083eed1f241a63 Mon Sep 17 00:00:00 2001
From: Bread Genie <msuhailbh07@gmail.com>
Date: Thu, 4 Jul 2024 11:42:26 +0530
Subject: [PATCH] feat(SiteDomains): show certbot error in a banner

---
 dashboard/src2/components/ObjectList.vue |  9 ++----
 dashboard/src2/objects/site.js           | 37 ++++++++++++++++++++++++
 press/press/doctype/site/site.py         |  8 +++++
 3 files changed, 48 insertions(+), 6 deletions(-)

diff --git a/dashboard/src2/components/ObjectList.vue b/dashboard/src2/components/ObjectList.vue
index 0d95a3c9f1..a2cd0cd004 100644
--- a/dashboard/src2/components/ObjectList.vue
+++ b/dashboard/src2/components/ObjectList.vue
@@ -1,11 +1,8 @@
 <template>
 	<div>
-		<AlertBanner
-			v-if="banner"
-			:title="banner.title"
-			:type="banner.type"
-			class="mb-4"
-		/>
+		<AlertBanner v-if="banner" v-bind="banner" class="mb-4">
+			<Button v-if="banner.button" v-bind="banner.button" class="ml-auto" />
+		</AlertBanner>
 		<div class="flex items-center justify-between">
 			<slot name="header-left" v-bind="context">
 				<div v-if="showControls" class="flex items-center space-x-2">
diff --git a/dashboard/src2/objects/site.js b/dashboard/src2/objects/site.js
index ea1953b4ee..c307a4680c 100644
--- a/dashboard/src2/objects/site.js
+++ b/dashboard/src2/objects/site.js
@@ -561,6 +561,43 @@ export default {
 							type: 'Badge'
 						}
 					],
+					banner({ documentResource: site }) {
+						if (site.doc.broken_domain_error) {
+							return {
+								title:
+									'There was an error fetching an https certificate for your domain.',
+								type: 'error',
+								button: {
+									label: 'View Error',
+									variant: 'outline',
+									onClick() {
+										renderDialog(
+											h(
+												GenericDialog,
+												{
+													options: {
+														title: 'Error fetching certificate',
+														size: 'xl'
+													}
+												},
+												{
+													default: () => {
+														return h('pre', {
+															class:
+																'whitespace-pre-wrap text-sm rounded border-2 p-3 border-gray-200 bg-gray-100',
+															innerHTML: site.doc.broken_domain_error
+														});
+													}
+												}
+											)
+										);
+									}
+								}
+							};
+						} else {
+							return null;
+						}
+					},
 					primaryAction({ listResource: domains, documentResource: site }) {
 						return {
 							label: 'Add Domain',
diff --git a/press/press/doctype/site/site.py b/press/press/doctype/site/site.py
index 10e117af49..55bef73df9 100644
--- a/press/press/doctype/site/site.py
+++ b/press/press/doctype/site/site.py
@@ -227,6 +227,14 @@ def get_doc(self, doc):
 		doc.server_title = server.title
 		doc.inbound_ip = self.inbound_ip
 		doc.is_dedicated_server = is_dedicated_server(self.server)
+
+		if broken_domain_tls_certificate := frappe.db.get_value(
+			"Site Domain", {"site": self.name, "status": "Broken"}, "tls_certificate"
+		):
+			doc.broken_domain_error = frappe.db.get_value(
+				"TLS Certificate", broken_domain_tls_certificate, "error"
+			)
+
 		return doc
 
 	def site_action(allowed_status: List[str]):