fix early bail when testing if string field should link to File node (#…

…6504) * fix early bail when for file inferring * remove any `application/octet-stream` handling to be in line with mime@2 behaviour * this can be shorter, doh!
gatsbyjs · Jul 18, 2018 · a8124b0 · a8124b0
1 parent 4159317
commit a8124b0
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/packages/gatsby-source-filesystem/src/create-file-node.js b/packages/gatsby-source-filesystem/src/create-file-node.js
@@ -44,7 +44,7 @@ exports.createFileNode = async (
     internal = {
       contentDigest,
       type: `File`,
-      mediaType: mediaType ? mediaType : `application/octet-stream`,
+      mediaType,
       description: `File "${path.relative(process.cwd(), slashed)}"`,
     }
   }

diff --git a/packages/gatsby-transformer-toml/src/gatsby-node.js b/packages/gatsby-transformer-toml/src/gatsby-node.js
@@ -5,7 +5,7 @@ const crypto = require(`crypto`)
 async function onCreateNode({ node, actions, loadNodeContent, createNodeId }) {
   const { createNode, createParentChildLink } = actions
   // Filter out non-toml content
-  // Currently TOML files are considered 'application/octet-stream' in 'mime-db'
+  // Currently TOML files are considered null in 'mime-db'
   // Hence the extension test instead of mediaType test
   if (node.extension !== `toml`) {
     return

diff --git a/packages/gatsby/src/schema/types/type-file.js b/packages/gatsby/src/schema/types/type-file.js
@@ -28,7 +28,7 @@ export function setFileNodeRootType(fileNodeRootType) {
 function pointsToFile(nodes, key, value) {
   const looksLikeFile =
     _.isString(value) &&
-    mime.getType(value) !== `application/octet-stream` &&
+    mime.getType(value) !== null &&
     // domains ending with .com
     mime.getType(value) !== `application/x-msdownload` &&
     isRelative(value) &&