Skip to content

Conversation

sandhose
Copy link
Member

  • syn2mas: refactor the metrics logic in the progress module
  • syn2mas: replace #[allow] annotations with #[expect]
  • Make a few password-related options public in the config crate
  • Move the synapse_idp_id field to the top of the provider section
  • Option to generate a MAS config from an existing Synapse config
  • syn2mas: add a buffered channel for writing threepids
  • syn2mas: add a buffered channel for writing external IDs
  • syn2mas: add a buffered channel for writing refreshable tokens
  • syn2mas: introduce a WriteBatch trait to refactor how we write to MAS
  • syn2mas: implement WriteBatch for MasNewUser
  • syn2mas: implement WriteBatch for MasNewUserPassword
  • syn2mas: implement WriteBatch for MasNewEmailThreepid
  • syn2mas: implement WriteBatch for MasNewUnsupportedThreepid
  • syn2mas: implement WriteBatch for MasNewUpstreamOauthLink
  • syn2mas: implement WriteBatch for MasNewCompatSession
  • syn2mas: implement WriteBatch for MasNewCompatAccessToken
  • syn2mas: implement WriteBatch for MasNewCompatRefreshToken
  • syn2mas: make the MasWriteBuffer use the WriteBatch trait
  • syn2mas: remove the MasWriter::write_ methods and replaced them in tests
  • syn2mas: reduce the channel buffer size
  • syn2mas: log the number of entities migrated at each step
  • syn2mas: only log once when rebuilding constraints
  • syn2mas: spawn the writer connections in parallel
  • syn2mas: warn about existing oauth-delegated user_external_ids
  • syn2mas: provide guidance on how to re-do a fresh migration
  • syn2mas: drop the experimental flag
  • Add a few missing license headers
  • Allow syn2mas arguments to be specified after the subcommand
  • syn2mas: log progress more often
  • syn2mas: introduce a dry-run mode

sandhose added 30 commits April 18, 2025 11:32
We don't need to carry around the various meters. Just make them global.
Also removes unused #[allow] annotations.
It also adds docs to a few of those options
This means that when serializing those, it will be at a more obvious place.
This is a best-effort conversion, which will warn about unsupported options.
Also make sure we have a single span for them
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Apr 23, 2025

Deploying matrix-authentication-service-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2539220
Status: ✅  Deploy successful!
Preview URL: https://ceef0a5e.matrix-authentication-service-docs.pages.dev
Branch Preview URL: https://quenting-syn2mas-polish.matrix-authentication-service-docs.pages.dev

View logs

@sandhose sandhose requested a review from reivilibre April 23, 2025 14:43
Copy link
Contributor

@reivilibre reivilibre left a comment

Choose a reason for hiding this comment

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

really nice to see this!

@sandhose sandhose merged commit 599fc85 into main Apr 23, 2025
21 checks passed
@sandhose sandhose deleted the quenting/syn2mas-polish branch April 23, 2025 15:01
@sandhose sandhose added A-Migration Related to the migration tooling T-Enhancement New feature of request labels Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Migration Related to the migration tooling T-Enhancement New feature of request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants