Use media-download directory everywhere in media-download service

guardian · Jan 6, 2025 · 9634016 · 9634016
1 parent e8a1d03
commit 9634016
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/packages/media-download/src/index.ts b/packages/media-download/src/index.ts
@@ -19,7 +19,7 @@ import {
 	MediaDownloadJob,
 } from '@guardian/transcription-service-common';
 
-const DOWNLOAD_DIRECTORY = '/media-download';
+export const MEDIA_DOWNLOAD_WORKING_DIRECTORY = '/media-download';
 
 const uploadToS3 = async (
 	s3Client: S3Client,
@@ -143,7 +143,7 @@ const main = async () => {
 
 	const metadata = await downloadMedia(
 		job.url,
-		DOWNLOAD_DIRECTORY,
+		MEDIA_DOWNLOAD_WORKING_DIRECTORY,
 		job.id,
 		proxyUrl,
 	);

diff --git a/packages/media-download/src/yt-dlp.ts b/packages/media-download/src/yt-dlp.ts
@@ -1,6 +1,7 @@
 import fs from 'node:fs';
 import { runSpawnCommand } from '@guardian/transcription-service-backend-common/src/process';
 import { logger } from '@guardian/transcription-service-backend-common';
+import { MEDIA_DOWNLOAD_WORKING_DIRECTORY } from './index';
 
 export type MediaMetadata = {
 	title: string;
@@ -26,7 +27,11 @@ export const startProxyTunnel = async (
 	port: number,
 ): Promise<string> => {
 	try {
-		fs.writeFileSync('/tmp/media_download', key + '\n', { mode: 0o600 });
+		fs.writeFileSync(
+			`${MEDIA_DOWNLOAD_WORKING_DIRECTORY}/media_download`,
+			key + '\n',
+			{ mode: 0o600 },
+		);
 		const result = await runSpawnCommand(
 			'startProxyTunnel',
 			'ssh',
@@ -42,7 +47,7 @@ export const startProxyTunnel = async (
 				'-N',
 				'-f',
 				'-i',
-				'/tmp/media_download',
+				`${MEDIA_DOWNLOAD_WORKING_DIRECTORY}/media_download`,
 				`media_download@${ip}`,
 			],
 			true,