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

Commit ee98b67

Browse files
committed
Update syn2mas advisor to reflect latest MAS capabilities
1 parent 91f1c7a commit ee98b67

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

tools/syn2mas/src/advisor.mts

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -125,8 +125,8 @@ export async function advisor(): Promise<void> {
125125
);
126126
}
127127
if (synapseConfig.enable_registration_captcha) {
128-
error(
129-
"Synapse config has registration CAPTCHA enabled which isn't supported by MAS: https://github.com/matrix-org/matrix-authentication-service/issues/138",
128+
warn(
129+
"Synapse config has registration CAPTCHA enabled which will need to be configured in MAS",
130130
);
131131
}
132132
if (synapseConfig.user_consent) {
@@ -209,10 +209,6 @@ export async function advisor(): Promise<void> {
209209
warn(
210210
"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",
211211
);
212-
} else if (synapseConfig.password_config?.enabled !== false) {
213-
warn(
214-
"Synapse has password auth enabled, but support for password auth in MAS is not feature complete",
215-
);
216212
}
217213

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

0 commit comments

Comments
 (0)