Saturday, 2024-07-13

noonedeadpunk> django docs say this should be stripped from incoming requests at the lb, added only for incoming requests at the lb which were https10:40
noonedeadpunkI _think_ we're doing that?10:40
noonedeadpunkjust LB does redirect to https10:41
noonedeadpunkthough, I guess that it might be easier for us not to handle logic for SECURE_PROXY_SSL_HEADER...10:41
noonedeadpunkyeah, so I'm also kind of confused about the chain we have...10:42
noonedeadpunkand kinda... not sure how to sort that except start debugging what django does recieve in each case10:42
noonedeadpunkas these headers should be used only when _all_ conditions are met... otherwise left as Nne10:43
jrossernoonedeadpunk: we maybe could state that it’s not supported to visit the horizon backend with a browser12:02
jrosserthat would start to reduce the complexity as we would have haproxy vip as the entry point always, for internal and external12:03
jrosserit feels like there is still code in the horizon role to support ssl termination at Apache and clients using that directly12:06
jrosserwe could start by removing that and relying on haproxy to insert the needed  headers12:06

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!