coop-cloud/traefik
8
2
Fork 18
Code Issues 12 Pull Requests 6 Releases Wiki Activity

fix: matrix-federation entrypoint on :8448 instead of :9001 #105

Open
notplants wants to merge 4 commits from compose.matrix into master
pull from: compose.matrix
merge into: coop-cloud:master
Conversation 6 Commits 4 Files Changed 2 +2 -2
notplants commented 2026-05-25 16:50:44 +00:00
Owner
Copy Link

compose.matrix.yml was mapping port 9001, but it probably should have been mapping port 8448 (which is the default matrix expects).

I saw that federation was not working before via this, and after making the change, matrix federation appears to be working from https://federationtester.matrix.org

compose.matrix.yml was mapping port 9001, but it probably should have been mapping port 8448 (which is the default matrix expects). I saw that federation was not working before via this, and after making the change, matrix federation appears to be working from https://federationtester.matrix.org
notplants added 1 commit 2026-05-25 16:50:45 +00:00
fix: matrix-federation entrypoint on :8448 to match published port
Some checks failed
continuous-integration/drone/pr Build is failing
Details
e94da63371 
compose.matrix.yml publishes container port 8448, but the entrypoint
was on :9001 — federation traffic hit no listener. :9001 also collided
with the minio-console entrypoint.
notplants force-pushed compose.matrix from e94da63371 to 88f1e61da2 2026-05-25 16:53:49 +00:00 Compare
notplants requested review from p4u1 2026-05-25 16:55:46 +00:00
notplants requested review from decentral1se 2026-05-25 16:55:56 +00:00
notplants requested review from javielico 2026-05-25 16:56:05 +00:00
notplants requested review from moritz 2026-05-25 16:56:13 +00:00
p4u1 commented 2026-06-03 16:25:58 +00:00
Owner
Copy Link

looks good to me, but could you revert the version bump in compose.yml? Also is there any other matrix maintainer that could review this?

looks good to me, but could you revert the version bump in compose.yml? Also is there any other matrix maintainer that could review this?
notplants commented 2026-06-03 17:42:25 +00:00
Author
Owner
Copy Link

ty for the review @p4u1 . if I revert the version bump, will I still be able to apply this update to traefik via upgrade? or would I have to use --chaos to get the update?

and @3wordchant reviewed the matrix PR here

although the cooperative.computer deployment of matrix is having an issue today, which I'm looking into

ty for the review @p4u1 . if I revert the version bump, will I still be able to apply this update to traefik via upgrade? or would I have to use --chaos to get the update? and @3wordchant reviewed the matrix PR [here](https://git.coopcloud.tech/coop-cloud/matrix-synapse/pulls/62) although the cooperative.computer deployment of matrix is having an issue today, which I'm looking into
p4u1 commented 2026-06-05 07:41:26 +00:00
Owner
Copy Link

Ok, just ping when this is ready to merge :)

You don't need --chaos to deploy this without a released version. You can always set the version of their recipe to the commit hash you want to deploy. This also works with commits on a different branch which is really neat!

TYPE=traefik:<commit hash>
Ok, just ping when this is ready to merge :) You don't need --chaos to deploy this without a released version. You can always set the version of their recipe to the commit hash you want to deploy. This also works with commits on a different branch which is really neat! ``` TYPE=traefik:<commit hash> ```
❤️ 1
notplants added 1 commit 2026-06-10 17:14:22 +00:00
Merge branch 'master' into compose.matrix
Some checks failed
continuous-integration/drone/pr Build is failing
Details
793901a7c2
notplants added 1 commit 2026-06-10 17:15:29 +00:00
Revert version number to 5.1.1
Some checks failed
continuous-integration/drone/pr Build is failing
Details
488a656631
notplants commented 2026-06-10 17:16:46 +00:00
Author
Owner
Copy Link

@p4u1 that's good to know about reference by commit and different branches (ty).

Just reverted the version bump in the PR, and all the related work on the matrix recipe is merged in now too.

So I believe this is ready to be merged

@p4u1 that's good to know about reference by commit and different branches (ty). Just reverted the version bump in the PR, and all the related work on the matrix recipe is merged in now too. So I believe this is ready to be merged
❤️ 2
decentral1se approved these changes 2026-06-11 10:52:14 +00:00
decentral1se left a comment
Owner
Copy Link

LGTM! Can't test however. Thanks for the work!

LGTM! Can't test however. Thanks for the work!
👍 1
p4u1 approved these changes 2026-06-11 11:03:20 +00:00
Some checks failed
continuous-integration/drone/pr Build is failing
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin compose.matrix:compose.matrix
git checkout compose.matrix
Sign in to join this conversation.
Reviewers
No Reviewers
javielico
moritz
decentral1se
p4u1
Labels
Clear labels
bug
Something is not working
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
3wordchant aadil abra-bot ammaratef45 amras BornDeleuze Brooke carla cas coopcloud decentral1se fauno javielico jjsfunhouse kawaiipunk knoflook moosemower moritz notplants oxaliq p4u1 renovate-bot simon stevensting trav yksflip
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/traefik#105
No description provided.
Delete Branch "compose.matrix"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?