Skip to content

Commit f732d9d

Browse files
committed
move cert generation to an earlier bit
1 parent e7422b6 commit f732d9d

File tree

1 file changed

+12
-7
lines changed

1 file changed

+12
-7
lines changed

src/charm.py

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -857,12 +857,6 @@ def _on_start(self, event: StartEvent) -> None:
857857
if not self.unit.is_leader():
858858
return
859859

860-
if not self.get_secret(APP_SCOPE, Config.WebhookManager.CRT_SECRET) or not self.get_secret(
861-
APP_SCOPE, Config.WebhookManager.KEY_SECRET
862-
):
863-
cert, key = gen_certificate(SERVICE_NAME, self.model.name)
864-
self.set_secret(APP_SCOPE, Config.WebhookManager.CRT_SECRET, cert.decode())
865-
self.set_secret(APP_SCOPE, Config.WebhookManager.KEY_SECRET, key.decode())
866860
self._initialise_replica_set(event)
867861
try:
868862
self._initialise_users(event)
@@ -1384,6 +1378,17 @@ def _check_or_set_keyfile(self) -> None:
13841378
if not self.get_secret(APP_SCOPE, "keyfile"):
13851379
self._generate_keyfile()
13861380

1381+
def _check_or_set_webhook_certs(self) -> None:
1382+
"""Set TLS certs for webhooks."""
1383+
if not self.unit.is_leader():
1384+
return
1385+
if not self.get_secret(APP_SCOPE, "webhook-certificate") or not self.get_secret(
1386+
APP_SCOPE, "webhook-key"
1387+
):
1388+
cert, key = gen_certificate(Config.WebhookManager.SERVICE_NAME, self.model.name)
1389+
self.set_secret(APP_SCOPE, "webhook-certificate", cert.decode())
1390+
self.set_secret(APP_SCOPE, "webhook-key", key.decode())
1391+
13871392
def _generate_keyfile(self) -> None:
13881393
self.set_secret(APP_SCOPE, "keyfile", generate_keyfile())
13891394

@@ -1410,8 +1415,8 @@ def _generate_secrets(self) -> None:
14101415
"""
14111416
self._check_or_set_user_password(OperatorUser)
14121417
self._check_or_set_user_password(MonitorUser)
1413-
14141418
self._check_or_set_keyfile()
1419+
self._check_or_set_webhook_certs()
14151420

14161421
def _initialise_replica_set(self, event: StartEvent) -> None:
14171422
"""Initialise replica set and create users."""

0 commit comments

Comments
 (0)