MediaTypePlugin.fileMatcher should take the URI

jdesrosiers · jdesrosiers · commit 6186d2f1a9e5 · 2024-08-07T11:54:26.000-07:00
... as opposed to the os-dependent file system path.
diff --git a/lib/uri-schemes/file-scheme-plugin.js b/lib/uri-schemes/file-scheme-plugin.js
@@ -22,7 +22,7 @@ const retrieve = async (uri, baseUri) => {
     responseUri = pathToFileURL(await readlink(filePath)).toString();
   }
 
-  const contentType = await getFileMediaType(filePath);
+  const contentType = await getFileMediaType(responseUri);
   const stream = createReadStream(filePath);
   const response = new Response(stream, {
     headers: { "Content-Type": contentType }

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ const retrieve = async (uri, baseUri) => {`
`22`	`22`	`responseUri = pathToFileURL(await readlink(filePath)).toString();`
`23`	`23`	`}`
`24`	`24`
`25`		`- const contentType = await getFileMediaType(filePath);`
	`25`	`+ const contentType = await getFileMediaType(responseUri);`
`26`	`26`	`const stream = createReadStream(filePath);`
`27`	`27`	`const response = new Response(stream, {`
`28`	`28`	`headers: { "Content-Type": contentType }`