feat: Update reverse proxy host format in IngressController

Update the format of the reverse proxy host in the IngressController to remove the port number if it is 80. This ensures consistency and improves readability of the code.
This commit is contained in:
Théophile Diot 2024-08-14 15:35:54 +01:00
parent ae1004f10e
commit f58a433308
No known key found for this signature in database
GPG key ID: FA995104A0BA376A

View file

@ -65,7 +65,7 @@ class IngressController(Controller):
if not pod:
self._logger.warning(f"Missing container bunkerweb in pod {controller_instance.metadata.name}")
else:
elif pod.env:
for env in pod.env:
instance["env"][env.name] = env.value or ""
@ -118,7 +118,10 @@ class IngressController(Controller):
self._logger.warning(f"Ignoring ingress rule with service {path.backend.service.name} : service not found.")
continue
reverse_proxy_host = f"http://{path.backend.service.name}.{namespace}.svc.cluster.local:{path.backend.service.port.number}"
reverse_proxy_host = f"http://{path.backend.service.name}.{namespace}.svc.cluster.local"
if path.backend.service.port.number != 80:
reverse_proxy_host += f":{path.backend.service.port.number}"
service.update(
{
"USE_REVERSE_PROXY": "yes",