Remove storybook (github#21037)

* Remove storybook * Update index.js
Anthonysidesapp · Aug 23, 2021 · 86b6b1a · 86b6b1a
1 parent d84b873
commit 86b6b1a
Show file tree

Hide file tree

Showing 12 changed files with 18,245 additions and 35,867 deletions.
diff --git a/.eslintignore b/.eslintignore
diff --git a/.gitignore b/.gitignore
@@ -10,7 +10,6 @@ coverage/
 /data/early-access
 .next
 .eslintcache
-/storybook/
 
 # blc: broken link checker
 blc_output.log

diff --git a/.storybook/main.js b/.storybook/main.js
diff --git a/.storybook/preview.js b/.storybook/preview.js
diff --git a/Dockerfile b/Dockerfile
@@ -48,7 +48,6 @@ COPY content/index.md ./content/index.md
 COPY next.config.js ./next.config.js
 COPY tsconfig.json ./tsconfig.json
 COPY next-env.d.ts ./next-env.d.ts
-COPY .storybook ./.storybook
 
 RUN npx tsc --noEmit
 

diff --git a/components/stories/Callout.stories.tsx b/components/stories/Callout.stories.tsx
diff --git a/components/stories/Introduction.stories.mdx b/components/stories/Introduction.stories.mdx
diff --git a/middleware/csp.js b/middleware/csp.js
@@ -70,10 +70,5 @@ export default function csp(req, res, next) {
     csp.directives.scriptSrc.push("'unsafe-inline'")
   }
 
-  if (req.path.startsWith('/storybook')) {
-    csp.directives.scriptSrc.push("'unsafe-eval'", "'unsafe-inline'")
-    csp.directives.frameSrc.push("'self'")
-  }
-
   return contentSecurityPolicy(csp)(req, res, next)
 }
diff --git a/middleware/index.js b/middleware/index.js
@@ -1,5 +1,4 @@
 import express from 'express'
-import basicAuth from 'express-basic-auth'
 import instrument from '../lib/instrument-middleware.js'
 import haltOnDroppedConnection from './halt-on-dropped-connection.js'
 import abort from './abort.js'
@@ -62,7 +61,6 @@ import renderPage from './render-page.js'
 const { NODE_ENV } = process.env
 const isDevelopment = NODE_ENV === 'development'
 const isTest = NODE_ENV === 'test' || process.env.GITHUB_ACTIONS === 'true'
-const isProduction = NODE_ENV === 'production' && process.env.HEROKU_PRODUCTION_APP
 
 // Catch unhandled promise rejections and passing them to Express's error handler
 // https://medium.com/@Abazhenov/using-async-await-in-express-with-node-8-b8af872c0016
@@ -145,18 +143,6 @@ export default function (app) {
       instrument(archivedEnterpriseVersionsAssets, './archived-enterprise-versions-assets')
     )
   )
-  app.use('/storybook', [
-    (isProduction &&
-      basicAuth({ users: { octocat: process.env.STORYBOOK_PASSWORD }, challenge: true })) ||
-      ((req, res, next) => next()),
-    express.static('storybook', {
-      index: false,
-      etag: false,
-      immutable: true,
-      lastModified: false,
-      maxAge: '1 day', // Relatively short in case we update index.html
-    }),
-  ])
   app.use(
     '/assets',
     express.static('assets', {