allow using ; as RDN separator according to rfc2253

[cpuschma]

Fixes #351

[johnweldon]

We need to fix the failing legacy tests.. (not in this PR) I think we accidentally introduced code depending on new language / library features that are not available in Go 1.9

[cpuschma]

I agree, but the error why the Go 1.9 build fails seems to come from go-ntlmssp:

Error: ../../../go/src/github.com/Azure/go-ntlmssp/negotiator.go:37:34: req.Header.Values undefined (type http.Header has no field or method Values)
Error: ../../../go/src/github.com/Azure/go-ntlmssp/negotiator.go:63:34: res.Header.Values undefined (type http.Header has no field or method Values)
Error: ../../../go/src/github.com/Azure/go-ntlmssp/negotiator.go:78:34: res.Header.Values undefined (type http.Header has no field or method Values)
Error: ../../../go/src/github.com/Azure/go-ntlmssp/negotiator.go:115:34: res.Header.Values undefined (type http.Header has no field or method Values)
Error: Process completed with exit code 2.

Looking at their travis.yml, they only run tests against 1.10.x and the latest version of Go :/

[shaiacs]

@johnweldon any plans for a release with this fix?

[johnweldon]

@johnweldon any plans for a release with this fix?

I'll try to get to this in the next few days

[shaiacs]

@johnweldon sorry to be nagging about this, but any chance you might get around to releasing this fix?

[johnweldon]

@johnweldon sorry to be nagging about this, but any chance you might get around to releasing this fix?

Done.... sorry for the delay.

[shaiacs]

No worries. Really appreciate this and all the work you've done here.