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

Federation now in compose #176

Merged
merged 4 commits into from
Feb 7, 2025
Merged

Federation now in compose #176

merged 4 commits into from
Feb 7, 2025

Conversation

pm-McFly
Copy link
Collaborator

@pm-McFly pm-McFly commented Feb 7, 2025

No description provided.

- ADDITIONAL_FEATURES=true
- FEDERATED_IDENTITY_SERVICES=fed.docker.localhost

Check notice

Code scanning / devskim

Accessing localhost could indicate debug code, or could hinder scaling. Note

Do not leave debug code in production
- fed.server
environment:
# - NODE_OPTIONS="--inspect=0.0.0.0:9229"
- BASE_URL=https://fed.docker.localhost

Check notice

Code scanning / devskim

Accessing localhost could indicate debug code, or could hinder scaling. Note

Do not leave debug code in production
environment:
# - NODE_OPTIONS="--inspect=0.0.0.0:9229"
- BASE_URL=https://fed.docker.localhost
- SERVER_NAME=docker.localhost

Check notice

Code scanning / devskim

Accessing localhost could indicate debug code, or could hinder scaling. Note

Do not leave debug code in production
@@ -148,6 +155,7 @@
- chat.docker.localhost
- matrix.docker.localhost
- tom.docker.localhost
- fed.docker.localhost

Check notice

Code scanning / devskim

Accessing localhost could indicate debug code, or could hinder scaling. Note

Do not leave debug code in production
@pm-McFly pm-McFly requested a review from ericlinagora February 7, 2025 16:59
@pm-McFly pm-McFly changed the title feat(fed): allow the use of trusted proxies Federation now in compose Feb 7, 2025
@pm-McFly pm-McFly merged commit 4ac8066 into master Feb 7, 2025
9 checks passed
@ericlinagora ericlinagora deleted the fed-express-config branch February 8, 2025 04:55
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