MoJo2600 / pihole-kubernetes

PiHole on kubernetes
503 stars 175 forks source link

fix: invalid web service manifest generation #288

Closed jpflouret closed 7 months ago

jpflouret commented 7 months ago

The pihole-web service is now valid when both serviceWeb.http and serviceWeb.https are disabled. Without this change, setting both of these to false will result in an invalid service manifest with an empty spec.ports value.

Description of the change

This change prevents the creation of the web service manifest if both serviceWeb.http.enabled and serviceWeb.https.enabled are false. Setting either of these values to true (the default) will preserve existing behaviour.

Benefits

Allows the user to not expose the pihole web service by disabling both http and https in the serviceWeb configuration option.

Possible drawbacks

Applicable issues

Additional information

Checklist

MoJo2600 commented 7 months ago

Thank you very much! LGTM!