posix: Evaluate UNC path support.

[harshavardhana]

https://blog.klauspost.com/long-windows-paths-unc-paths-in-go/

import "strings"
import "regexp"

// Pattern to match a windows absolute path: "c:\" and similar
var isAbsWinDrive = regexp.MustCompile(`^[a-zA-Z]\:\\`)

// UncPath converts an absolute Windows path
// to a UNC long path.
func UncPath(s string) string {
    // UNC can NOT use "/", so convert all to "\"
    s = strings.Replace(s, `/`, `\`, -1)

    // If prefix is "\\", we already have a UNC path or server.
    if strings.HasPrefix(s, `\\`) {

        // If already long path, just keep it
        if strings.HasPrefix(s, `\\?\`) {
            return s
        }

        // Trim "\\" from path and add UNC prefix.
        return "\\\\?\UNC\\" + strings.TrimPrefix(s, `\\`)
    }

    if isAbsWinDrive.MatchString(s) {
        return `\\?\` + s
    }

    return s
}

If it is any consolation, dot net users have the same issue - https://github.com/dotnet/corefx/issues/645

[harshavardhana]

This is a bug to track future support on this particular behavior - golang/go#10900

[harshavardhana]

In future support this - https://github.com/golang/sys/windows

[balamurugana]

See also golang/go#3358

[harshavardhana]

See also golang/go#3358

We need to get to a point somewhere in future to support it and also possibly contribute upstream eventually.

[balamurugana]

UNC path support needs to be added to golang core libraries to address this issue. For more information, see golang/go#3358

[balamurugana]

As per our discussion, we don't work with UNC path till there is a golang support. This issue can be closed.

[harshavardhana]

As per our discussion, we don't work with UNC path till there is a golang support. This issue can be closed.

Removing this bug from standalone server release keeping this bug for future work.

[harshavardhana]

Took the liberty of fixing it, and it works - fixed by #1887

[harshavardhana]

#1887 - 260 path limit on windows is fixed, now we have 32K path limit - unit tests have been updated as well. Please review the patch.

[abperiasamy]

Does os.Remove and similar functions support UNC path argument?

[harshavardhana]

Does os.Remove and similar functions support UNC path argument?

Yes they all do.