Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added nginx option to server requirements #139

Closed
wants to merge 1 commit into from

Conversation

steveroot
Copy link

I almost skipped YOURLS as an option as it only said Apache, but then I spotted a suggested config for Nginx. I haven't tested that, so this is just a link to that config so other new readers can read further

I almost skipped YOURLS as an option as it only said Apache, but then I spotted a suggested config for Nginx. I haven't tested that, so this is just a link to that config so other new readers can read further
@steveroot steveroot requested a review from a team as a code owner September 7, 2023 15:40
@LeoColomb
Copy link
Member

Thanks a lot for your pull request @steveroot.
That being said, I'd prefer to avoid mentioning Nginx here:

  • The main reason for keeping Apache httpd only is that YOURLS comes with an automatic setup for Apache server.
  • Nginx is not a requirement per se, and this list is only about requirements. Sure we are talking about http servers, but as soon as one is mentioned the other is unneeded.
  • And finally, the users who are skilled enough to manage Nginx configuration (tricker than Apache httpd), sure know that Nginx is the typical alternative.

I hope that makes sense 🙂

@LeoColomb LeoColomb closed this Sep 14, 2023
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.

2 participants