Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

Commit

Permalink
Update syn2mas advisor to reflect latest MAS capabilities
Browse files Browse the repository at this point in the history
  • Loading branch information
hughns committed Sep 6, 2024
1 parent 91f1c7a commit ee98b67
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions tools/syn2mas/src/advisor.mts
Original file line number Diff line number Diff line change
Expand Up @@ -125,8 +125,8 @@ export async function advisor(): Promise<void> {
);
}
if (synapseConfig.enable_registration_captcha) {
error(
"Synapse config has registration CAPTCHA enabled which isn't supported by MAS: https://github.com/matrix-org/matrix-authentication-service/issues/138",
warn(
"Synapse config has registration CAPTCHA enabled which will need to be configured in MAS",
);
}
if (synapseConfig.user_consent) {
Expand Down Expand Up @@ -209,10 +209,6 @@ export async function advisor(): Promise<void> {
warn(
"Synapse has a non-standard password auth enabled which won't work after migration and will need to be manually mapped to an upstream OpenID Provider during migration",
);
} else if (synapseConfig.password_config?.enabled !== false) {
warn(
"Synapse has password auth enabled, but support for password auth in MAS is not feature complete",
);
}

const externalIdAuthProviders = (await synapse
Expand All @@ -231,7 +227,7 @@ export async function advisor(): Promise<void> {
);
if (usersWithPassword > 0) {
warn(
`Synapse database contains ${usersWithPassword} users with a password which will be migrated. However, support for password auth in MAS is not feature complete`,
`Synapse database contains ${usersWithPassword} users with a password which will be migrated`,
);
}

Expand Down

0 comments on commit ee98b67

Please sign in to comment.