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

Add lemmyexternalproxy to lemmy in docker-compose.yml #2960

Merged
merged 2 commits into from
Jun 8, 2023

Conversation

jason-watkins
Copy link
Contributor

Consensus on the matrix channel is that this is necessary to let the lemmy container talk to the internet directly, which is in turn necessary for federation.

@jason-watkins
Copy link
Contributor Author

It's possible that the same result could be achieve with some more clever proxying by the nginx container, but I don't know exactly what would be needed to accomplish that.

@dessalines
Copy link
Member

This looks good / is fine to me, but this is also the dev docker setup linked. The production one(s) are really in the lemmy-ansible repo: https://github.com/LemmyNet/lemmy-ansible/tree/main/templates

@dessalines dessalines enabled auto-merge (squash) June 8, 2023 11:21
@dessalines dessalines merged commit 491e197 into LemmyNet:main Jun 8, 2023
@jason-watkins
Copy link
Contributor Author

Interesting. I didn't realize the manual docker instructions were only intended for development. The ansible playbook doesn't seem to support Arch, so I am using docker manually for that reason. Adding Arch support to the ansible setup is on my todo list, but not all that high (and it's right after "learn what the hell ansible is anyway" :))

@dessalines
Copy link
Member

Its not docker installs are intended only for dev, its just that the docker-compose.yml templates in the lemmy-ansible repo are the production ones, while the ones in the main branch in our lemmy git repo, are for development.

@Nutomic
Copy link
Member

Nutomic commented Jun 8, 2023

Originally the docker-compose.yml for prod was in the Lemmy repo, I guess the docs dont reflect that yet?

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.

3 participants