chore: replace deprecated String.prototype.substr()

.substr() is deprecated so we replace it with .slice() which works similarily but isn't deprecated Signed-off-by: Tobias Speicher <[email protected]>
webpack · Mar 18, 2022 · 5d99468 · 5d99468
1 parent 188497a
commit 5d99468
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/client-src/utils/createSocketURL.js b/client-src/utils/createSocketURL.js
@@ -5,7 +5,7 @@
 function format(objURL) {
   let protocol = objURL.protocol || "";
 
-  if (protocol && protocol.substr(-1) !== ":") {
+  if (protocol && !protocol.endsWith(":")) {
     protocol += ":";
   }
 

diff --git a/client-src/utils/parseURL.js b/client-src/utils/parseURL.js
@@ -9,7 +9,7 @@ function parseURL(resourceQuery) {
   let options = {};
 
   if (typeof resourceQuery === "string" && resourceQuery !== "") {
-    const searchParams = resourceQuery.substr(1).split("&");
+    const searchParams = resourceQuery.slice(1).split("&");
 
     for (let i = 0; i < searchParams.length; i++) {
       const pair = searchParams[i].split("=");

diff --git a/test/e2e/range-header.test.js b/test/e2e/range-header.test.js
@@ -42,7 +42,7 @@ describe("'Range' header", () => {
     );
     expect(responseRange.headers["content-length"]).toBe("500");
     expect(responseRange.headers["content-range"]).toMatch(/^bytes 0-499\//);
-    expect(responseRange.text).toBe(responseContent.substr(0, 500));
+    expect(responseRange.text).toBe(responseContent.slice(0, 500));
     expect(responseRange.text.length).toBe(500);
   });