FoundationEssentials: handle canonical and non-canonical paths (#668)

compnerd · web-flow · commit 50d7537507d5 · 2024-06-07T09:20:18.000-07:00
In the case that the path is canonicalised, the `/` would be replaced
with `\`. The leading `\` would interfere with our ability to recognise
the absolute path representation and thus get the wrong response.
diff --git a/Sources/FoundationEssentials/String/String+Internals.swift b/Sources/FoundationEssentials/String/String+Internals.swift
@@ -29,7 +29,7 @@ extension String {
         }
 
         var iter = self.utf8.makeIterator()
-        let bLeadingSlash = if iter.next() == ._slash, iter.next()?.isLetter ?? false, iter.next() == ._colon { true } else { false }
+        let bLeadingSlash = if [._slash, ._backslash].contains(iter.next()), iter.next()?.isLetter ?? false, iter.next() == ._colon { true } else { false }
 
         // Strip the leading `/` on a RFC8089 path (`/[drive-letter]:/...` ).  A
         // leading slash indicates a rooted path on the drive for the current

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ extension String {`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`var iter = self.utf8.makeIterator()`
`32`		`- let bLeadingSlash = if iter.next() == ._slash, iter.next()?.isLetter ?? false, iter.next() == ._colon { true } else { false }`
	`32`	`+ let bLeadingSlash = if [._slash, ._backslash].contains(iter.next()), iter.next()?.isLetter ?? false, iter.next() == ._colon { true } else { false }`
`33`	`33`
`34`	`34`	// Strip the leading `/` on a RFC8089 path (`/[drive-letter]:/...` ). A
`35`	`35`	`// leading slash indicates a rooted path on the drive for the current`