Added ApplyDefaultCommandsToQuerylessUrls route option.

imazen · Aug 11, 2023 · b444710 · b444710
1 parent e2a80fd
commit b444710
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 3 deletions.
diff --git a/src/Imageflow.Server.Configuration/Executor.cs b/src/Imageflow.Server.Configuration/Executor.cs
@@ -121,7 +121,10 @@ public ImageflowMiddlewareOptions GetImageflowMiddlewareOptions(){
             }      
 
         }
-
+        if (config.RouteDefaults?.ApplyDefaultCommandsToQuerylessUrls ?? false){
+            options.SetApplyDefaultCommandsToQuerylessUrls(true);
+        }
+
         // set security options
         var securityOptions = new SecurityOptions();
         if (config.Security?.MaxDecodeResolution != null){

diff --git a/src/Imageflow.Server.Configuration/MinimalParseStructures.cs b/src/Imageflow.Server.Configuration/MinimalParseStructures.cs
@@ -101,6 +101,7 @@ internal class RouteBase : ITomlMetadataProvider, IValidationCapable
    // public RouteSource? Source { get; set; }
     public string? CacheControl { get; set; }
     public string? ApplyDefaultCommands { get; set; }
+    public bool? ApplyDefaultCommandsToQuerylessUrls { get; set; }
     // public string? ApplyOverrideCommands { get; set; }
     // public string? RequireSignature { get; set; }
 

diff --git a/src/Imageflow.Server.Configuration/minimal.toml b/src/Imageflow.Server.Configuration/minimal.toml
@@ -11,6 +11,7 @@ lowercase_path_remainder = false
 allow_extensionless_urls = false
 cache_control = "public, max-age=2592000"
 apply_default_commands = "quality=76&webp.quality=70&f.sharpen=23&down.filter=mitchell"
+apply_default_commands_to_queryless_urls = false
 
 [[routes]]
 prefix = '/images/'

diff --git a/src/Imageflow.Server/ImageJobInfo.cs b/src/Imageflow.Server/ImageJobInfo.cs
@@ -219,7 +219,7 @@ private bool ProcessRewritesAndAuthorization(HttpContext context, ImageflowMiddl
             }
 
             // Set defaults if keys are missing, but at least 1 supported key is present
-            if (PathHelpers.SupportedQuerystringKeys.Any(args.Query.ContainsKey))
+            if (middlewareOptions.ApplyDefaultCommandsToQuerylessUrls || PathHelpers.SupportedQuerystringKeys.Any(args.Query.ContainsKey))
             {
                 foreach (var pair in middlewareOptions.CommandDefaults)
                 {

diff --git a/src/Imageflow.Server/ImageflowMiddlewareOptions.cs b/src/Imageflow.Server/ImageflowMiddlewareOptions.cs
@@ -55,7 +55,8 @@ public class ImageflowMiddlewareOptions
 
 
         internal readonly Dictionary<string, string> CommandDefaults = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase);
-
+        public bool ApplyDefaultCommandsToQuerylessUrls { get; set; } = false;
+
         internal readonly Dictionary<string, PresetOptions> Presets = new Dictionary<string, PresetOptions>(StringComparer.OrdinalIgnoreCase);
 
         internal readonly List<ExtensionlessPath> ExtensionlessPaths = new List<ExtensionlessPath>();
@@ -242,5 +243,16 @@ public ImageflowMiddlewareOptions SetDefaultCacheControlString(string cacheContr
             DefaultCacheControlString = cacheControlString;
             return this;
         }
+
+        /// <summary>
+        /// Set to true have the command defaults to all urls, not just ones containing a valid query command.
+        /// </summary>
+        /// <param name="value"></param>
+        /// <returns></returns>
+        public ImageflowMiddlewareOptions SetApplyDefaultCommandsToQuerylessUrls(bool value)
+        {
+            this.ApplyDefaultCommandsToQuerylessUrls = value;
+            return this;
+        }
     }
 }