Fix invite text and add accounting context to translation messages #53

					
				@ -57,3 +57,3 @@

				        c["domain"] = site.domain

				        c["site_name"] = site.name

				        c["protocol"] = "http" if self.request and not self.request.is_secure() else "https"

				        c["protocol"] = "https"  # if self.request and not self.request.is_secure() else "https"

reynir commented

2024-08-15 07:41:17 +00:00

I think with our current nginx proxy setup we can check first for X-Forwarded-Proto header and otherwise do that. It may be brittle to check that header...

I think with our current nginx proxy setup we can check first for `X-Forwarded-Proto` header and otherwise do that. It may be brittle to check that header...

benjaoming commented

2024-08-15 14:23:07 +00:00

You're right, it turns out the correct method is to define HOW the fact of https presence should be detected:

https://docs.djangoproject.com/en/5.0/ref/settings/#secure-proxy-ssl-header

You're right, it turns out the correct method is to define HOW the fact of https presence should be detected: https://docs.djangoproject.com/en/5.0/ref/settings/#secure-proxy-ssl-header

🎉 1

reynir commented

2024-08-15 14:34:16 +00:00

Ah awesome! Well spotted