search

DOMjudge / domjudge-packaging

DOMjudge packaging for (Linux) distributions and live image
31 stars 37 forks source link

Feat add trusted proxies #102

Closed Dup4 closed 2 years ago

Dup4 commented 2 years ago

https://github.com/DOMjudge/domjudge/issues/1368#issuecomment-979286836

Allow the user to pass in TRUSTED_PROXIES through environment variables when starting the container, and we will append the TRUSTED_PROXIES passed in by the user together.

Dup4 commented 2 years ago

Added Nginx related trusted proxies configuration.

https://github.com/nickygerritsen/domjudge/blob/4c404d3ed1363cd7c261105ed505978d1ffdf9bd/etc/nginx-conf-inner.in#L57-L65

nickygerritsen commented 2 years ago

Thanks, looks good now. I will push a new 7.3.4 image when CI is done and I have merged this PR. But I think you used main so you can do it yourself 😄

nickygerritsen commented 2 years ago

@Dup4 I just saw you added TRUSTED_PROXIES to the wrong place in the readme: it is in the judgehost part instead of the server.