From 168fbc65eec7540719a4d62c8ef95152cb35fcef Mon Sep 17 00:00:00 2001
From: Rhys Arkins <rhys@arkins.net>
Date: Wed, 14 Aug 2019 11:26:03 +0200
Subject: [PATCH] fix(docker-compose): detect null config, warn only for
parsing errors
---
lib/manager/docker-compose/extract.ts | 14 ++++++++++++--
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/lib/manager/docker-compose/extract.ts b/lib/manager/docker-compose/extract.ts
index 510c539211..b003478255 100644
--- a/lib/manager/docker-compose/extract.ts
+++ b/lib/manager/docker-compose/extract.ts
@@ -36,13 +36,23 @@ class LineMapper {
}
}
-export function extractPackageFile(content: string): PackageFile | null {
+export function extractPackageFile(
+ content: string,
+ fileName?: string
+): PackageFile | null {
logger.debug('docker-compose.extractPackageFile()');
let config: DockerComposeConfig;
try {
config = safeLoad(content);
+ // istanbul ignore if
+ if (!config) {
+ logger.info(
+ { fileName, content },
+ 'Null config when parsing Docker Compose content'
+ );
+ }
} catch (err) {
- logger.error({ err }, 'Parsing Docker Compose config YAML');
+ logger.warn({ fileName, err }, 'Parsing Docker Compose config YAML');
return null;
}
const lineMapper = new LineMapper(content, /^\s*image:/);
--
GitLab