fix: should allow srcset on source tag

vitejs · Apr 10, 2021 · 88a0132 · 88a0132
1 parent fe0e754
commit 88a0132
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/packages/vite/src/node/plugins/asset.ts b/packages/vite/src/node/plugins/asset.ts
@@ -257,6 +257,11 @@ export async function urlToBuiltUrl(
   const file = url.startsWith('/')
     ? path.join(config.root, url)
     : path.join(path.dirname(importer), url)
+  const stats = await fsp.stat(file)
+  if (stats.isDirectory()) {
+    return url
+  }
+
   return fileToBuiltUrl(
     file,
     config,

diff --git a/packages/vite/src/node/plugins/html.ts b/packages/vite/src/node/plugins/html.ts
@@ -69,7 +69,7 @@ export function htmlInlineScriptProxyPlugin(): Plugin {
 export const assetAttrsConfig: Record<string, string[]> = {
   link: ['href'],
   video: ['src', 'poster'],
-  source: ['src'],
+  source: ['src', 'srcset'],
   img: ['src', 'srcset'],
   image: ['xlink:href', 'href'],
   use: ['xlink:href', 'href']

diff --git a/packages/vite/src/node/utils.ts b/packages/vite/src/node/utils.ts
@@ -383,7 +383,7 @@ export async function processSrcSet(
         .split(' ', 2)
       return { url, descriptor }
     })
-    .filter(({ url, descriptor }) => url && descriptor)
+    .filter(({ url }) => !!url)
 
   const ret = await Promise.all(
     imageCandidates.map(async ({ url, descriptor }) => {