fix(github): compare repo names lower case (#19411)

Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
renovatebot · Dec 20, 2022 · a0fe269 · a0fe269
1 parent fc37657
commit a0fe269
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 3 deletions.
diff --git a/lib/modules/platform/github/index.ts b/lib/modules/platform/github/index.ts
@@ -34,7 +34,7 @@ import * as hostRules from '../../../util/host-rules';
 import * as githubHttp from '../../../util/http/github';
 import { regEx } from '../../../util/regex';
 import { sanitize } from '../../../util/sanitize';
-import { fromBase64 } from '../../../util/string';
+import { fromBase64, looseEquals } from '../../../util/string';
 import { ensureTrailingSlash } from '../../../util/url';
 import type {
   AggregatedVulnerabilities,
@@ -719,7 +719,7 @@ export async function findPr({
       return false;
     }
 
-    if (!config.forkToken && config.repository !== p.sourceRepo) {
+    if (!config.forkToken && !looseEquals(config.repository, p.sourceRepo)) {
       return false;
     }
 

diff --git a/lib/util/string.spec.ts b/lib/util/string.spec.ts
@@ -1,4 +1,4 @@
-import { replaceAt } from './string';
+import { looseEquals, replaceAt } from './string';
 
 describe('util/string', () => {
   describe('replaceAt', () => {
@@ -24,4 +24,12 @@ describe('util/string', () => {
       expect(newContent).toBe('I want to have a new pet maybe a dog');
     });
   });
+
+  describe('looseEquals', () => {
+    test('reverts to literal match if either is falsey', () => {
+      expect(looseEquals(undefined, null)).toBeFalse();
+      expect(looseEquals(null, null)).toBeTrue();
+      expect(looseEquals(null, '')).toBeFalse();
+    });
+  });
 });
diff --git a/lib/util/string.ts b/lib/util/string.ts
@@ -45,3 +45,13 @@ export function uniqueStrings(
 ): boolean {
   return elements.indexOf(element) === index;
 }
+
+export function looseEquals(
+  a: string | null | undefined,
+  b: string | null | undefined
+): boolean {
+  if (!(a && b)) {
+    return a === b;
+  }
+  return a.localeCompare(b, undefined, { sensitivity: 'base' }) === 0;
+}