Merge pull request #478 from Financial-Times/captions-logger

Add logging when appending captions
Financial-Times · Nov 29, 2023 · dcf53e6 · dcf53e6
2 parents 42f4abb + 15b5745
commit dcf53e6
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/server/lib/download/video.js b/server/lib/download/video.js
@@ -72,7 +72,14 @@ module.exports = exports = class VideoDownload extends ArticleDownload {
 
 				this.captionFiles = captionFiles;
 
-				captionFiles.forEach(({ file, name }) => this.append(file, { name }));
+				captionFiles.forEach(({ file, name }) => {
+					log.info('appendCaptions', {
+						name,
+						file
+					});
+
+					return this.append(file, { name })
+				});
 
 			}
 			catch (error) {