Tendrl / api

Tendrl API
GNU Lesser General Public License v2.1
16 stars 16 forks source link

Running Grafana behind a reverse proxy #447

Closed GowthamShanmugam closed 6 years ago

GowthamShanmugam commented 6 years ago

bugzilla: 1590416 tendrl-bug-id: Tendrl/api#446

Signed-off-by: GowthamShanmugam gshanmug@redhat.com

GowthamShanmugam commented 6 years ago

@shtripat @shirshendu @gnehapk @mbukatov please review

GowthamShanmugam commented 6 years ago

Yes http will come, if will show like http://{ip}/grafana/{dashboard_name}

shtripat commented 6 years ago

Yes http will come, if will show like http://{ip}/grafana/{dashboard_name}

So this ideally means we are not actually enabling https for grafana, rather we are redirecting https URL to http URL of grafana. Right?

So this means this is actually partially solved, as ideal situation would be to enable https for grafana as well if https enabled for tendrl server.

shirshendu commented 6 years ago

So this ideally means we are not actually enabling https for grafana, rather we are redirecting https URL to http URL of grafana. Right?

No, a reverse proxy is not redirection. The traffic is routed by apache over local loopback to grafana, because we're using 127.0.0.1. Local loopback is not susceptible to an attacker sniffing the traffic, unless the attacker already has access to the machine.

So this means this is actually partially solved, as ideal situation would be to enable https for grafana as well if https enabled for tendrl server.

No, the problem is solved completely for our current use-case. Only in case of separate grafana server would we need https to grafana, which (as we discussed in last meeting), is not a currently supported scenario. Anyone needing such a deployment will also need to configure grafana HTTPS, which can be advised on a case-by-case basis, or in documentation for SSL configuration.

shtripat commented 6 years ago

Ack

GowthamShanmugam commented 6 years ago

Tendrl/ui/pull/1074

GowthamShanmugam commented 6 years ago

@shirshendu @shtripat please review this and merge