From 9237274093b3f6c9fa2f69e92704e734366a461f Mon Sep 17 00:00:00 2001
From: Scott Cooper <scttcper@gmail.com>
Date: Sun, 5 Jul 2020 18:47:00 -0700
Subject: [PATCH] feat: use cloudflare js cdn, use all minified css and js
 BREAKING CHANGE: uses cloudflare cdn

---
 README.md                              |   3 +--
 lib/favicon-32x32.png                  | Bin 607 -> 0 bytes
 lib/{favicon-16x16.png => favicon.png} | Bin
 lib/index.hbs                          |   9 ++++-----
 lib/index.ts                           |  22 ++++++----------------
 package.json                           |   2 +-
 test/index.spec.ts                     |  16 +++++-----------
 7 files changed, 17 insertions(+), 35 deletions(-)
 delete mode 100644 lib/favicon-32x32.png
 rename lib/{favicon-16x16.png => favicon.png} (100%)

diff --git a/README.md b/README.md
index 9f32416..eb7baa6 100644
--- a/README.md
+++ b/README.md
@@ -42,8 +42,7 @@ routePrefix: '/docs', // route where the view is returned
 specPrefix: '/docs/spec', // route where the spec is returned
 exposeSpec: false, // expose spec file
 hideTopbar: false, // hide swagger top bar
-favicon16: '/favicon-16x16.png', // default icon 16x16, set for self icon
-favicon32: '/favicon-32x32.png', // default icon 32x32, set for self icon
+favicon: '/favicon.png', // default favicon
 ```
 
 ## example
diff --git a/lib/favicon-32x32.png b/lib/favicon-32x32.png
deleted file mode 100644
index 8b20f26da72b744fd860b71adbd9908308b36e20..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 607
zcmeAS@N?(olHy`uVBq!ia0vp^3LwnE3?yBabR7dyy#YQUu0Wc>z(T;l0fHn9JR~eU
zfTV~Cm;q!15s;0{g$M#k2P8EP9tuF@;h+)VU=ZP8kl<jD;a~wo1r81k79JfI4iydo
z6D%TDSR`z)$k<^}aKfPVm2SfegN6qN6*mkjE*KPCF!+7<@Cu-N#7lzwf*CjjBoquh
z0xBvxX3SWz;=qju7ykYG@!`5<5>N$WmZytjNCjhSaA;Gj0neLa@sb5QtiSKS>(#v3
z$bPENnHwEvUn+XfQajmyxoz#n-7gL9Xz(asZz#L1`&3*uE=r6k^_A29Gi!3<KYU7R
zx^p<&`qcMH6Ggd~v~7|9qH4FI|HRHM%0JzX1^MxuknZB|<j{@3Box!ieD`<VRvAD2
zjiSqc+S$IneNyz1;6Z=3Q1-@vLvOnmbXPwV=)Z7CEPX-l4Z$f3TeBui7u6{|S&&lR
z{OY))y5qwmChy)ny&#%q*tk^SZl&SL<%bT=7OGJx*(Z7Qb<_5xxn@g`uI`FXJ^ipt
zqjcHVmSs=&th3OUzN}YVtT3TlYu@ex!LTPu96^E0SxR||d^XP#NZ`wyH$jl`tH$~R
z?M}HGrw%z;3%&AOd2@6AG-I>V?q6!n7DZNjt@svN81y*fNX*5&GS^LWmhPFu!*hGj
ze@V-2*Z)gIH7?FuvU($XukAyHzZ)%>c8mXuur2z0O#ZZY-8uWU|JBc&DJ=T+|HocX
OO7V2{b6Mw<&;$VBc>Cl4

diff --git a/lib/favicon-16x16.png b/lib/favicon.png
similarity index 100%
rename from lib/favicon-16x16.png
rename to lib/favicon.png
diff --git a/lib/index.hbs b/lib/index.hbs
index 55f162e..1681a86 100644
--- a/lib/index.hbs
+++ b/lib/index.hbs
@@ -5,9 +5,8 @@
   <meta http-equiv="X-UA-Compatible" content="IE=edge">
   <title>{{title}}</title>
   <link href="https://fonts.googleapis.com/css?family=Open+Sans:400,700|Source+Code+Pro:300,600|Titillium+Web:400,600,700" rel="stylesheet">
-  <link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/swagger-ui-dist@{{swaggerVersion}}/swagger-ui.css">
-  <link rel="icon" type="image/png" href="{{favicon32}}" sizes="32x32" />
-  <link rel="icon" type="image/png" href="{{favicon16}}" sizes="16x16" />
+  <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/swagger-ui/{{swaggerVersion}}/swagger-ui.min.css">
+  <link rel="icon" type="image/png" href="{{favicon}}" />
   <style>
     html
     {
@@ -67,8 +66,8 @@
 
 <div id="swagger-ui"></div>
 
-<script src="https://cdn.jsdelivr.net/npm/swagger-ui-dist@{{swaggerVersion}}/swagger-ui-bundle.js"></script>
-<script src="https://cdn.jsdelivr.net/npm/swagger-ui-dist@{{swaggerVersion}}/swagger-ui-standalone-preset.js"></script>
+<script src="https://cdnjs.cloudflare.com/ajax/libs/swagger-ui/{{swaggerVersion}}/swagger-ui-bundle.min.js"></script>
+<script src="https://cdnjs.cloudflare.com/ajax/libs/swagger-ui/{{swaggerVersion}}/swagger-ui-standalone-preset.js"></script>
 <script>
 function HideTopbarPlugin() {
   // this plugin overrides the Topbar component to return nothing
diff --git a/lib/index.ts b/lib/index.ts
index acae58e..cc2e81a 100644
--- a/lib/index.ts
+++ b/lib/index.ts
@@ -27,8 +27,7 @@ export interface KoaSwaggerUiOptions {
   specPrefix: string;
   exposeSpec: boolean;
   hideTopbar: boolean;
-  favicon16: string;
-  favicon32: string;
+  favicon: string;
 }
 
 const defaultOptions: KoaSwaggerUiOptions = {
@@ -44,8 +43,7 @@ const defaultOptions: KoaSwaggerUiOptions = {
   swaggerVersion: '',
   exposeSpec: false,
   hideTopbar: false,
-  favicon16: '/favicon-16x16.png',
-  favicon32: '/favicon-32x32.png',
+  favicon: '/favicon.png',
 };
 
 function koaSwagger(config: Partial<KoaSwaggerUiOptions> = {}): Middleware {
@@ -59,10 +57,8 @@ function koaSwagger(config: Partial<KoaSwaggerUiOptions> = {}): Middleware {
   }
 
   // Setup icons
-  const extFavicon16 = config.favicon16;
-  const extFavicon32 = config.favicon32;
-  const favicon16Path = join(__dirname, defaultOptions.favicon16);
-  const favicon32Path = join(__dirname, defaultOptions.favicon32);
+  const extFavicon = config.favicon;
+  const faviconPath = join(__dirname, defaultOptions.favicon);
 
   // Setup default options
   const options: KoaSwaggerUiOptions = defaultsDeep(config, defaultOptions);
@@ -86,15 +82,9 @@ function koaSwagger(config: Partial<KoaSwaggerUiOptions> = {}): Middleware {
       return true;
     }
 
-    if (extFavicon16 === undefined && ctx.path === defaultOptions.favicon16) {
+    if (extFavicon === undefined && ctx.path === defaultOptions.favicon) {
       ctx.type = 'image/png';
-      ctx.body = createReadStream(favicon16Path);
-      return true;
-    }
-
-    if (extFavicon32 === undefined && ctx.path === defaultOptions.favicon32) {
-      ctx.type = 'image/png';
-      ctx.body = createReadStream(favicon32Path);
+      ctx.body = createReadStream(faviconPath);
       return true;
     }
 
diff --git a/package.json b/package.json
index 5b6b744..206157e 100644
--- a/package.json
+++ b/package.json
@@ -13,7 +13,7 @@
     "lint:fix": "eslint --ext .ts,.js --fix .",
     "prepare": "npm run build",
     "build": "del-cli dist && tsc -p tsconfig.build.json",
-    "postbuild": "cp lib/index.hbs dist && cp lib/favicon-32x32.png dist && cp lib/favicon-16x16.png dist",
+    "postbuild": "cp lib/index.hbs dist && cp lib/favicon.png dist",
     "test": "jest",
     "test:watch": "jest --watch",
     "test:ci": "jest --ci --coverage --no-cache"
diff --git a/test/index.spec.ts b/test/index.spec.ts
index 3b64909..82f2c9d 100644
--- a/test/index.spec.ts
+++ b/test/index.spec.ts
@@ -14,24 +14,18 @@ describe('koa2-swagger-ui', () => {
   it('should return css', async () => {
     const result = readPkgUp.sync({ cwd: __dirname });
     const version = result!.packageJson.devDependencies!['swagger-ui-dist'];
-    const url = `https://cdn.jsdelivr.net/npm/swagger-ui-dist@${version}`;
+    const url = `https://cdnjs.cloudflare.com/ajax/libs/swagger-ui/${version}`;
     await request(url)
-      .get('/swagger-ui.css')
-      .expect('Content-Type', 'text/css; charset=utf-8')
+      .get('/swagger-ui.min.css')
+      .expect('Content-Type', 'text/css')
       .expect(200);
   });
   it('should return spec', async () => {
     await request(app.callback()).get('/docs/spec').expect('Content-Type', /json/).expect(200);
   });
-  it('should return icon16x16', async () => {
+  it('should return favicon', async () => {
     await request(app.callback())
-      .get('/favicon-16x16.png')
-      .expect('Content-Type', /png/)
-      .expect(200);
-  });
-  it('should return icon32x32', async () => {
-    await request(app.callback())
-      .get('/favicon-32x32.png')
+      .get('/favicon.png')
       .expect('Content-Type', /png/)
       .expect(200);
   });