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

Changing default reverse proxy config to work locally #1040

Closed
wants to merge 2 commits into from
Closed

Changing default reverse proxy config to work locally #1040

wants to merge 2 commits into from

Conversation

NeutroGe
Copy link

@NeutroGe NeutroGe commented Jul 1, 2024

Hello, i think most users run jellyfin on the same local server as they run their reverse nginx proxy, so it seems logical that the nginx config files are set this way by default instead of using the dns resolve mechanism as it currently does, so i'm proposing this small change to reflect this. It should allow people to get their reverse proxy working more easily and less troubleshooting questions about it.

Hello, i think most users run jellyfin on the same local server as they run their reverse nginx proxy, so it seems logical that the nginx config files are set this way by default instead of using the dns resolve mechanism as it currently does, so i'm proposing this small change to reflect this. It should allow people to get their reverse proxy working more easily and less troubleshooting questions about it.
@felix920506
Copy link
Member

Please provide a title more descriptive of the changes made.

@NeutroGe NeutroGe changed the title Update nginx.md Changing default reverse proxy config to work locally Jul 1, 2024
@NeutroGe
Copy link
Author

NeutroGe commented Jul 1, 2024

Please provide a title more descriptive of the changes made.

done!

docs/general/networking/nginx.md Show resolved Hide resolved
docs/general/networking/nginx.md Show resolved Hide resolved
Copy link
Member

@joshuaboniface joshuaboniface left a comment

Choose a reason for hiding this comment

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

I like the change and agree with the idea but second @felix920506's comment about suggesting people change this to something else if needed.

@jellyfin-bot
Copy link

Cloudflare Pages deployment

Latest commit 6b5c1bd9dee9a2a4549c91950d77aaf8b91d6368
Status ✅ Deployed!
Preview URL https://aeb012c2.jellyfin-org.pages.dev
Type 🔀 Preview

@NeutroGe
Copy link
Author

So much nonsense for such a small change, last time i try to help here

@NeutroGe NeutroGe closed this Aug 23, 2024
@NeutroGe NeutroGe deleted the patch-1 branch August 23, 2024 20:28
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.

4 participants