search

canonical / catalogue-k8s-operator

A service catalogue for workloads running in Juju.
https://charmhub.io/catalogue-k8s
0 stars 3 forks source link

[Feature request] Expose config option to enable https redirect #127

Open baycarbone opened 2 months ago

baycarbone commented 2 months ago

Bug Description

Although the charm has the ability to enable https redirect through the traefik ingress library, the ability to actually enable it is not surfaced to the administrator of the charm. It would be great if the administrator of the charm could enable https redirect through a config option.

see:

To Reproduce

Install COS by following: https://charmhub.io/topics/canonical-observability-stack/tutorials/install-microk8s And use the TLS overlay as explained here: https://charmhub.io/topics/canonical-observability-stack/explanation/tls Browse to the catalogue endpoint over http: you won't be redirected to https.

Environment

Deployment as part of the COS bundle with TLS overlay

Relevant log output

n/a

Additional context

This was also discussed here before opening this issue.

baycarbone commented 2 months ago

As mentioned in https://github.com/canonical/cos-lite-bundle/issues/114#issuecomment-2255021911, this will not be made configurable but automatically enabled like in the other COS charms.

lucabello commented 1 month ago

We should add redirect_https=True (ref) to our IngressPerAppRequirer.