fix: update regex for overlay path and text to allow additional safe characters

Author: manu4543

src/url/builder.ts

@@ -2,8 +2,8 @@ import { ImageKitOptions, UrlOptions } from "../interfaces";
 import { ImageOverlay, SolidColorOverlay, SubtitleOverlay, TextOverlay, Transformation, VideoOverlay } from "../interfaces/Transformation";
 import transformationUtils, { safeBtoa } from "../utils/transformation";
 const TRANSFORMATION_PARAMETER = "tr";
-const SIMPLE_OVERLAY_PATH_REGEX = new RegExp('^[a-zA-Z0-9-._,/ ]*$')
-const SIMPLE_OVERLAY_TEXT_REGEX = new RegExp('^[a-zA-Z0-9-._, ]*$') // These characters are selected by testing actual URLs on both path and query parameters. If and when backend starts supporting wide range of characters, this regex should be updated to improve URL readability.
+const SIMPLE_OVERLAY_PATH_REGEX = new RegExp('^[a-zA-Z0-9-._/ ]*$')
+const SIMPLE_OVERLAY_TEXT_REGEX = new RegExp('^[a-zA-Z0-9-._ ]*$') // These characters are selected by testing actual URLs on both path and query parameters. If and when backend starts supporting wide range of characters, this regex should be updated to improve URL readability.
 
 function removeTrailingSlash(str: string) {
   if (typeof str == "string" && str[str.length - 1] == "/") {

test/url-generation/overlay.js

@@ -318,17 +318,17 @@ describe("Overlay encoding test cases", function () {
         expect(url).equal(`https://ik.imagekit.io/demo/tr:l-text,i-Manu,l-end/medium_cafe_B1iTdD0C.jpg`);
     });
 
-    it('Simple text overlay with spaces and comma, should use i instead of ie', function () {
+    it('Simple text overlay with spaces and other safe characters, should use i instead of ie', function () {
         const url = imagekit.url({
             path: "/medium_cafe_B1iTdD0C.jpg",
             transformation: [{
                 overlay: {
                     type: "text",
-                    text: "alnum123-._, ",
+                    text: "alnum123-._ ",
                 }
             }]
         });
-        expect(url).equal(`https://ik.imagekit.io/demo/tr:l-text,i-${encodeURIComponent("alnum123-._, ")},l-end/medium_cafe_B1iTdD0C.jpg`);
+        expect(url).equal(`https://ik.imagekit.io/demo/tr:l-text,i-${encodeURIComponent("alnum123-._ ")},l-end/medium_cafe_B1iTdD0C.jpg`);
     });
 
     it('Non simple text overlay, should use ie instead of i', function () {