From 980cf00cbaeba44bf5734956d3ed51d992498981 Mon Sep 17 00:00:00 2001
From: Mole <nikolajlauridsen@protonmail.ch>
Date: Wed, 1 Dec 2021 19:53:27 +0100
Subject: [PATCH] V9: Fix backoffice javascript loading slowly (#11714)

* Update Smidge

* Add SmidgeRequestHelper to override the compression order

By default Smidge will always try and use Brotli (br), this compression is good in the the sense that it compresses the file quite a bit, but sadly it's super super slow, like 10 seconds for backoffice js slow.

* Remove unnecessary new keyword

Co-authored-by: nikolajlauridsen <nel@umbraco.dk>
---
 .../UmbracoBuilderExtensions.cs               |  2 +
 .../SmidgeRequestHelper.cs                    | 78 +++++++++++++++++++
 .../Umbraco.Web.Common.csproj                 |  4 +-
 3 files changed, 82 insertions(+), 2 deletions(-)
 create mode 100644 src/Umbraco.Web.Common/RuntimeMinification/SmidgeRequestHelper.cs

diff --git a/src/Umbraco.Web.Common/DependencyInjection/UmbracoBuilderExtensions.cs b/src/Umbraco.Web.Common/DependencyInjection/UmbracoBuilderExtensions.cs
index fda4d7ca32..84538c9310 100644
--- a/src/Umbraco.Web.Common/DependencyInjection/UmbracoBuilderExtensions.cs
+++ b/src/Umbraco.Web.Common/DependencyInjection/UmbracoBuilderExtensions.cs
@@ -275,6 +275,8 @@ namespace Umbraco.Extensions
             });
 
             builder.Services.AddSmidge(builder.Config.GetSection(Constants.Configuration.ConfigRuntimeMinification));
+            // Replace the Smidge request helper, in order to discourage the use of brotli since it's super slow
+            builder.Services.AddUnique<IRequestHelper, SmidgeRequestHelper>();
             builder.Services.AddSmidgeNuglify();
             builder.Services.AddSmidgeInMemory(false); // it will be enabled based on config/cachebuster
 
diff --git a/src/Umbraco.Web.Common/RuntimeMinification/SmidgeRequestHelper.cs b/src/Umbraco.Web.Common/RuntimeMinification/SmidgeRequestHelper.cs
new file mode 100644
index 0000000000..4313e0e359
--- /dev/null
+++ b/src/Umbraco.Web.Common/RuntimeMinification/SmidgeRequestHelper.cs
@@ -0,0 +1,78 @@
+using System.Collections.Generic;
+using Microsoft.AspNetCore.Http;
+using Microsoft.Extensions.Primitives;
+using Microsoft.Net.Http.Headers;
+using Smidge;
+using Smidge.Models;
+
+namespace Umbraco.Cms.Web.Common.RuntimeMinification
+{
+    public class SmidgeRequestHelper : IRequestHelper
+    {
+        private RequestHelper _wrappedRequestHelper;
+
+        public SmidgeRequestHelper(IWebsiteInfo siteInfo)
+        {
+            _wrappedRequestHelper = new RequestHelper(siteInfo);
+        }
+
+        /// <inheritdoc/>
+        public string Content(string path) => _wrappedRequestHelper.Content(path);
+
+        /// <inheritdoc/>
+        public string Content(IWebFile file) => _wrappedRequestHelper.Content(file);
+
+        /// <inheritdoc/>
+        public bool IsExternalRequestPath(string path) => _wrappedRequestHelper.IsExternalRequestPath(path);
+
+        /// <summary>
+        /// Overrides the default order of compression from Smidge, since Brotli is super slow (~10 seconds for backoffice.js)
+        /// </summary>
+        /// <param name="headers"></param>
+        /// <returns></returns>
+        public CompressionType GetClientCompression(IDictionary<string, StringValues> headers)
+        {
+            var type = CompressionType.None;
+
+            if (headers is not IHeaderDictionary headerDictionary)
+            {
+                headerDictionary = new HeaderDictionary(headers.Count);
+                foreach ((var key, StringValues stringValues) in headers)
+                {
+                    headerDictionary[key] = stringValues;
+                }
+            }
+
+            var acceptEncoding = headerDictionary.GetCommaSeparatedValues(HeaderNames.AcceptEncoding);
+            if (acceptEncoding.Length > 0)
+            {
+                // Prefer in order: GZip, Deflate, Brotli.
+                for (var i = 0; i < acceptEncoding.Length; i++)
+                {
+                    var encoding = acceptEncoding[i].Trim();
+
+                    CompressionType parsed = CompressionType.Parse(encoding);
+
+                    // Not pack200-gzip.
+                    if (parsed == CompressionType.GZip)
+                    {
+                        return CompressionType.GZip;
+                    }
+
+                    if (parsed == CompressionType.Deflate)
+                    {
+                        type = CompressionType.Deflate;
+                    }
+
+                    // Brotli is typically last in the accept encoding header.
+                    if (type != CompressionType.Deflate && parsed == CompressionType.Brotli)
+                    {
+                        type = CompressionType.Brotli;
+                    }
+                }
+            }
+
+            return type;
+        }
+    }
+}
diff --git a/src/Umbraco.Web.Common/Umbraco.Web.Common.csproj b/src/Umbraco.Web.Common/Umbraco.Web.Common.csproj
index 1f03b83bf1..031a7a782f 100644
--- a/src/Umbraco.Web.Common/Umbraco.Web.Common.csproj
+++ b/src/Umbraco.Web.Common/Umbraco.Web.Common.csproj
@@ -35,8 +35,8 @@
       <PackageReference Include="NETStandard.Library" Version="2.0.3" />
       <PackageReference Include="Serilog.AspNetCore" Version="4.1.0" />
       <PackageReference Include="SixLabors.ImageSharp.Web" Version="1.0.4" />
-      <PackageReference Include="Smidge.Nuglify" Version="4.0.2" />
-      <PackageReference Include="Smidge.InMemory" Version="4.0.2" />
+      <PackageReference Include="Smidge.Nuglify" Version="4.0.3" />
+      <PackageReference Include="Smidge.InMemory" Version="4.0.3" />
       <PackageReference Include="Dazinator.Extensions.FileProviders" Version="2.0.0" />
       <PackageReference Include="Umbraco.Code" Version="1.2.0">
         <PrivateAssets>all</PrivateAssets>