Skip to content

Updates Expected Synapse Version to be Supported - #336

Merged
pm-McFly merged 1 commit into
devfrom
chore/update-synapse-version
May 29, 2026
Merged

Updates Expected Synapse Version to be Supported#336
pm-McFly merged 1 commit into
devfrom
chore/update-synapse-version

Conversation

@pm-McFly

Copy link
Copy Markdown
Collaborator

No description provided.

@pm-McFly pm-McFly self-assigned this Mar 17, 2026
@coderabbitai

coderabbitai Bot commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: ddd1336e-e770-4855-88ac-a45756e63f0b

📥 Commits

Reviewing files that changed from the base of the PR and between 93e11b7 and 62eeb07.

📒 Files selected for processing (4)
  • compose.yml
  • packages/federated-identity-service/src/__testData__/docker-compose.yml
  • packages/matrix-identity-server/matrix-server/docker-compose.yml
  • packages/matrix-identity-server/matrix-server/init

📝 Walkthrough

Synapse Version Bump: v1.119.0 → v1.152.1

Fundamental Issue:
The project was running Synapse v1.119.0, which is severely outdated (33 minor versions behind). This introduces security vulnerabilities and compatibility gaps—hence the security/urgent labels.

Changes:
Updated Synapse Docker image tag across four deployment/test configurations:

  • compose.yml
  • packages/federated-identity-service/src/__testData__/docker-compose.yml
  • packages/matrix-identity-server/matrix-server/docker-compose.yml
  • packages/matrix-identity-server/matrix-server/init (initialization script)

Validation:
Unit tests pass. v1.152.1 has been tested and validated with ToM. Intermediate versions (v1.135.2, v1.139.2) have been staged and validated on QA/STG/LNG/TWP environments.

Technical Debt Acknowledged:
v1.140+ requires separate Dockerfile updates to sync the S3 connector (synapse-s3-storage-provider) to v1.6.0. This dependency update is not addressed in this PR.

Data Flow Impact:
None—this is purely an infrastructure/dependency version pin change with no application code modifications.

Walkthrough

The Synapse Docker image dependency is upgraded from matrixdotorg/synapse:v1.119.0 to matrixdotorg/synapse:v1.152.1 across four configuration points: the root Compose file, the federated identity service test template, the matrix identity server Compose file, and the initialization script.

Changes

Synapse v1.152.1 Dependency Update

Layer / File(s) Summary
Synapse image version bump across configurations
compose.yml, packages/federated-identity-service/src/__testData__/docker-compose.yml, packages/matrix-identity-server/matrix-server/docker-compose.yml, packages/matrix-identity-server/matrix-server/init
The matrixdotorg/synapse Docker image reference is bumped from v1.119.0 to v1.152.1 consistently across all Docker Compose files and the initialization script.

Suggested labels

dependencies, package::federation-server, package::identity-server


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nx-cloud

nx-cloud Bot commented Mar 17, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 53c5096

Command Status Duration Result
nx affected -t test ✅ Succeeded 1s View ↗
nx affected -t build ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-05-29 14:22:32 UTC

@pm-McFly

Copy link
Copy Markdown
Collaborator Author

@Crash-- FYI, unit tests are still good with newer synapse version, I see issue locally as well

I keep this as draft until we test it in a QA env to validate with many users and some usage history

@pm-McFly pm-McFly added security Security risk to users of the project. priority::urgent We want to do this as soon as possible. chore Modification that does not relate to a bug fix, a new feature or a documentation update labels Mar 18, 2026
@pm-McFly
pm-McFly force-pushed the dev branch 2 times, most recently from 5950f13 to fbcc854 Compare April 8, 2026 08:08
@pm-McFly

Copy link
Copy Markdown
Collaborator Author

@pm-McFly

Copy link
Copy Markdown
Collaborator Author

versions up to v1.152.1 tested and validated with ToM

@pm-McFly
pm-McFly force-pushed the chore/update-synapse-version branch from 9b5a35a to 53c5096 Compare May 29, 2026 14:08
Signed-off-by: Pierre 'McFly' Marty <pmarty@linagora.com>
@pm-McFly
pm-McFly force-pushed the chore/update-synapse-version branch from 53c5096 to 62eeb07 Compare May 29, 2026 14:19
@pm-McFly
pm-McFly marked this pull request as ready for review May 29, 2026 14:30
@pm-McFly
pm-McFly merged commit ceaa83b into dev May 29, 2026
9 of 11 checks passed
@pm-McFly
pm-McFly deleted the chore/update-synapse-version branch June 4, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Modification that does not relate to a bug fix, a new feature or a documentation update priority::urgent We want to do this as soon as possible. security Security risk to users of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant