fix: update retrieveBlueprintAsset to return a Promise and await its result in the assets route

Author: jstarpl

meteor/server/api/blueprints/api.ts

@@ -127,7 +127,7 @@ export async function uploadBlueprintAsset(cred: RequestCredentials, fileId: str
 	await fsp.mkdir(assetDirPath, { recursive: true })
 	await fsp.writeFile(assetPath, data)
 }
-export function retrieveBlueprintAsset(_cred: RequestCredentials, fileId: string): ReadStream {
+export async function retrieveBlueprintAsset(_cred: RequestCredentials, fileId: string): Promise<ReadStream> {
 	check(fileId, String)
 
 	const storePath = getSystemStorePath()
@@ -137,7 +137,11 @@ export function retrieveBlueprintAsset(_cred: RequestCredentials, fileId: string
 		throw new Error('Requested asset outside of asset storage path')
 	}
 
-	return createReadStream(assetPath)
+	const stream = createReadStream(assetPath)
+	return new Promise((resolve, reject) => {
+		stream.on('open', () => resolve(stream))
+		stream.on('error', (err) => reject(err))
+	})
 }
 /** Only to be called from internal functions */
 export async function internalUploadBlueprint(

meteor/server/api/blueprints/http.ts

@@ -187,7 +187,7 @@ blueprintsRouter.get('/assets/*fileId', async (ctx) => {
 	const filePath = ctx.params.fileId
 	if (filePath.match(/\.(png|svg|gif)?$/)) {
 		try {
-			const dataStream = retrieveBlueprintAsset(ctx, filePath)
+			const dataStream = await retrieveBlueprintAsset(ctx, filePath)
 			const extension = path.extname(filePath)
 			if (extension === '.svg') {
 				ctx.response.type = 'image/svg+xml'