Skip to content

Commit f4d07ad

Browse files
committed
Remove obsolete OIDC provider warning
We now produce an error for unmapped OIDC providers!
1 parent 945f314 commit f4d07ad

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

crates/syn2mas/src/synapse_reader/checks.rs

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -80,9 +80,6 @@ pub enum CheckError {
8080
/// proceeding with the migration.
8181
#[derive(Debug, Error)]
8282
pub enum CheckWarning {
83-
#[error("Synapse config contains OIDC auth configuration (issuer: {issuer:?}) which will need to be manually mapped to an upstream OpenID Connect Provider during migration.")]
84-
UpstreamOidcProvider { issuer: String },
85-
8683
#[error("Synapse config contains {0} auth configuration which will need to be manually mapped as an upstream OAuth 2.0 provider during migration.")]
8784
ExternalAuthSystem(&'static str),
8885

@@ -112,15 +109,6 @@ pub fn synapse_config_check(synapse_config: &Config) -> (Vec<CheckWarning>, Vec<
112109
warnings.push(CheckWarning::DisableUserConsentAfterMigration);
113110
}
114111

115-
// TODO check the settings directly against the MAS settings
116-
for provider in synapse_config.all_oidc_providers().values() {
117-
if let Some(ref issuer) = provider.issuer {
118-
warnings.push(CheckWarning::UpstreamOidcProvider {
119-
issuer: issuer.clone(),
120-
});
121-
}
122-
}
123-
124112
// TODO provide guidance on migrating these
125113
if synapse_config.cas_config.enabled {
126114
warnings.push(CheckWarning::ExternalAuthSystem("CAS"));

0 commit comments

Comments
 (0)