fix(http-multipart): issue where IE 11 was stripping needed '/' from paths

Author: jonsamwell

CHANGELOG.md

@@ -1,9 +1,15 @@
+1.08
+========
+Fixed issue in IE11 that was stripping leading '/' on urls which in turned caused the routing in web api to fail.
+
 1.07
 ========
 Fixed deploy issue
+
 1.06
 ========
 Fixed deploy issue
+
 1.0.5
 ========
 Added ability to have fine grained control over the request that get batched by adding an optional canBatchRequest function on the configuration object.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ngx-http-batcher",
-  "version": "1.0.7",
+  "version": "1.0.8",
   "description": "Angular (2+) HTTP batching module to reduce the number of HTTP requests and increase performance",
   "scripts": {
     "build": "./node_modules/.bin/ngc -p ./src",

src/adapters/http-multipart-mixed-boundary-adapter.ts

@@ -143,10 +143,20 @@ export class HttpMultipartMixedBoundaryAdapter implements IBatchHttpRequestAdapt
   private getUrlParts(url: string): { host: string; path: string, search: string } {
     const anchorElement = document.createElement("a");
     anchorElement.href = url;
+    // IE < 11 & Safari 11 seem to remove the proceeding '/' form urls
+    // which causings routing errors in web api
+    // https://localhost:44379/authentication/profile/current should be
+    // path: "/authentication/profile/current"
+    // in IE it is
+    // path: "authentication/profile/current"
     return {
       host: anchorElement.host,
-      path: anchorElement.pathname,
+      path: this.ensureLeadingBackSlash(anchorElement.pathname),
       search: anchorElement.search
     };
   }
+
+  private ensureLeadingBackSlash(path: string): string {
+    return path[0] === "/" ? path : `/${path}`;
+  }
 }