Skip to content

Conversation

p-rintz
Copy link
Contributor

@p-rintz p-rintz commented May 23, 2025

Adding a simple, but complete example of an Nginx config for a reverse proxy.

Not sure if the federation port at 8448 is actually still needed.

@p-rintz p-rintz requested a review from a team as a code owner May 23, 2025 20:53
@CLAassistant
Copy link

CLAassistant commented May 23, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

dyff of changes in rendered templates of CI manifests

No changes in rendered templates

Copy link
Member

@gaelgatelement gaelgatelement left a comment

Choose a reason for hiding this comment

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

The configuration seems fine as far as I can tell. I saw 2 comments that would require fixing before merging. Thanks!

@p-rintz
Copy link
Contributor Author

p-rintz commented May 26, 2025

Removed both snippets.

@gaelgatelement
Copy link
Member

Thank you for your contribution!

@gaelgatelement gaelgatelement merged commit 303f264 into element-hq:main May 27, 2025
53 of 54 checks passed
@p-rintz p-rintz deleted the nginx-config branch May 28, 2025 10:50
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