Update trimming of paths so that it's compatible with non-root installs and tests pass

Rick Mason · Rick Mason · commit 525dc4e2f96e · 2017-06-20T09:29:20.000+01:00
diff --git a/src/UmbracoFileSystemProviders.Azure/AzureFileSystem.cs b/src/UmbracoFileSystemProviders.Azure/AzureFileSystem.cs
@@ -665,10 +665,10 @@ private string ResolveUrl(string path, bool relative)
 
             if (this.UseDefaultRoute)
             {
-                return $"{this.ApplicationVirtualPath?.Trim('/')}/{Constants.DefaultMediaRoute}/{fixedPath}";
+                return $"{this.ApplicationVirtualPath?.TrimEnd('/')}/{Constants.DefaultMediaRoute}/{fixedPath}";
             }
 
-            return $"{this.ApplicationVirtualPath?.Trim('/')}/{this.ContainerName}/{fixedPath}";
+            return $"{this.ApplicationVirtualPath?.TrimEnd('/')}/{this.ContainerName}/{fixedPath}";
         }
 
         /// <summary>

Original file line number	Diff line number	Diff line change
`@@ -665,10 +665,10 @@ private string ResolveUrl(string path, bool relative)`
`665`	`665`
`666`	`666`	`if (this.UseDefaultRoute)`
`667`	`667`	`{`
`668`		`- return $"{this.ApplicationVirtualPath?.Trim('/')}/{Constants.DefaultMediaRoute}/{fixedPath}";`
	`668`	`+ return $"{this.ApplicationVirtualPath?.TrimEnd('/')}/{Constants.DefaultMediaRoute}/{fixedPath}";`
`669`	`669`	`}`
`670`	`670`
`671`		`- return $"{this.ApplicationVirtualPath?.Trim('/')}/{this.ContainerName}/{fixedPath}";`
	`671`	`+ return $"{this.ApplicationVirtualPath?.TrimEnd('/')}/{this.ContainerName}/{fixedPath}";`
`672`	`672`	`}`
`673`	`673`
`674`	`674`	`/// <summary>`