coop-cloud/traefik
6
2
Fork 19
Code Issues 14 Pull Requests 7 Releases Wiki Activity

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

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

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
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
Copy Source

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
Copy Source

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
Copy Source

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
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
continuous-integration/drone/pr Build is failing
Details
488a656631
notplants commented 2026-06-10 17:16:46 +00:00
Author
Owner
Copy Link
Copy Source

@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
Copy Source

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
decentral1se commented 2026-07-01 10:52:59 +00:00
Owner
Copy Link
Copy Source

Sorry, let this one slip and now there is a conflict. I've merged it manually here:

81869a049e

Thanks @notplants!

Sorry, let this one slip and now there is a conflict. I've merged it manually here: > 81869a049e38853107c67a5edbc7d6efe845b5ce Thanks @notplants!
👍 1 ❤️ 1
decentral1se closed this pull request 2026-07-01 10:52:59 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
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 labels
Milestone
No items
No Milestone
Assignees
Clear assignees
3wordchant aadil (Aadil Ayub) abra-bot (Abra Bot) ammaratef45 amras (Sarma) BornDeleuze Brooke carla cas (Cassowary) coopcloud decentral1se (d1) fauno (fauno) javielico (Javielico) jjsfunhouse kawaiipunk (KawaiiPunk) knoflook moosemower moritz notplants oxaliq (sorrel) p4u1 renovate-bot (Comrade Renovate Bot) simon stevensting trav (Trav Fryer) yksflip
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/traefik#105