Make the namespace comparison in sources.rs case-insensitive (#643)

Fixed #629. The primary reason for this PR is documented there, as well.
EmbarkStudios · Mar 26, 2024 · bb7d86b · bb7d86b
1 parent 31720e5
commit bb7d86b
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/sources.rs b/src/sources.rs
@@ -104,11 +104,13 @@ pub fn check(ctx: crate::CheckCtx<'_, ValidConfig>, sink: impl Into<ErrorSink>)
             };
             get_org(url)
         }) {
+            // .to_lowercase() (ln. 113) enables case insensitivity, as GitHub and
+            // GitLab are case insensitive in regards to namespaces.
             if let Some(ind) = ctx
                 .cfg
                 .allowed_orgs
                 .iter()
-                .position(|(sorgt, sorgn)| orgt == *sorgt && sorgn.value.as_str() == orgname)
+                .position(|(sorgt, sorgn)| orgt == *sorgt && sorgn.value.as_str().to_lowercase() == orgname.to_lowercase())
             {
                 org_hits.as_mut_bitslice().set(ind, true);
                 diags::SourceAllowedByOrg {