fix(git-submodules): don’t include token in lookupName (#9556)

lib/manager/git-submodules/extract.spec.ts

 import { mock } from 'jest-mock-extended';
 import _simpleGit, { Response, SimpleGit } from 'simple-git';
 import { getName, partial } from '../../../test/util';
+import * as hostRules from '../../util/host-rules';
 import type { PackageFile } from '../types';
 import extractPackageFile from './extract';
 
@@ -44,6 +45,7 @@ describe(getName(__filename), () => {
   });
   describe('extractPackageFile()', () => {
     it('extracts submodules', async () => {
+      hostRules.add({ hostName: 'github.com', token: 'abc123' });
       let res: PackageFile;
       expect(
         await extractPackageFile('', '.gitmodules.1', { localDir })

lib/manager/git-submodules/extract.ts

@@ -114,7 +114,13 @@ export default async function extractPackageFile(
           // Find HTTP URL, then apply token
           let httpSubModuleUrl = getHttpUrl(subModuleUrl);
           const hostRule = hostRules.find({ url: httpSubModuleUrl });
-          httpSubModuleUrl = getHttpUrl(subModuleUrl, hostRule?.token);
+          if (hostRule?.token) {
+            logger.debug(
+              { httpSubModuleUrl },
+              'Found hostRules token for submodule'
+            );
+            httpSubModuleUrl = getHttpUrl(subModuleUrl, hostRule.token);
+          }
           const currentValue = await getBranch(
             gitModulesPath,
             name,
@@ -122,7 +128,7 @@ export default async function extractPackageFile(
           );
           return {
             depName: path,
-            lookupName: httpSubModuleUrl,
+            lookupName: getHttpUrl(subModuleUrl),
             currentValue,
             currentDigest,
           };