Skip to content

Conversation

vmcj
Copy link
Member

@vmcj vmcj commented Sep 14, 2025

This is highly opiniated but we have deployed with nginx for years and most new users will use nginx over apache2 currently (although they might even already use other webservers).

Most of our enhancements are added to the nginx config and not always added to the apache2 config also.

This should make support slightly easier in Slack as new users should be steered into a configuration we're more familiar with.

This is highly opiniated but we have deployed with nginx for years and
most new users will use nginx over apache2 currently (although they might even
already use other webservers).

Most of our enhancements are added to the nginx config and not always added to the
apache2 config also.
Copy link
Member

@eldering eldering left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No objections, but I didn't fully check the PR itself (whether all documentation was updated).

@vmcj
Copy link
Member Author

vmcj commented Sep 16, 2025

No objections, but I didn't fully check the PR itself (whether all documentation was updated).

I did a grep for nginx and those were the logical places for such documentation. We can do others if we find them.

@vmcj vmcj marked this pull request as ready for review September 16, 2025 05:47
@vmcj vmcj added this pull request to the merge queue Sep 19, 2025
Merged via the queue into DOMjudge:main with commit 1ce25b3 Sep 19, 2025
36 checks passed
@vmcj vmcj deleted the switch_apache2_nginx branch September 19, 2025 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants