Use golang's log, so we don't need log4go.

ntlm/crypto.go

@@ -8,8 +8,9 @@ import (
 	md5P "crypto/md5"
 	"crypto/rand"
 	rc4P "crypto/rc4"
-	md4P "github.com/ThomsonReutersEikon/go-ntlm/ntlm/md4"
 	crc32P "hash/crc32"
+
+	md4P "github.com/ThomsonReutersEikon/go-ntlm/ntlm/md4"
 )
 
 func md4(data []byte) []byte {

ntlm/helpers.go

@@ -5,8 +5,8 @@ package ntlm
 import (
 	"bytes"
 	"crypto/rand"
-	"unicode/utf16"
 	"encoding/binary"
+	"unicode/utf16"
 )
 
 // Concatenate two byte slices into a new slice
@@ -67,23 +67,22 @@ func utf16FromString(s string) []byte {
 
 // Convert a UTF16 string to UTF8 string for Go usage
 func utf16ToString(bytes []byte) string {
-       var data []uint16
+	var data []uint16
 
-       // NOTE: This is definitely not the best way to do this, but when I tried using a buffer.Read I could not get it to work
+	// NOTE: This is definitely not the best way to do this, but when I tried using a buffer.Read I could not get it to work
-       for offset := 0; offset < len(bytes); offset = offset + 2 {
+	for offset := 0; offset < len(bytes); offset = offset + 2 {
-               i := binary.LittleEndian.Uint16(bytes[offset : offset+2])
+		i := binary.LittleEndian.Uint16(bytes[offset : offset+2])
-               data = append(data, i)
+		data = append(data, i)
-       }
+	}
 
-       return string(utf16.Decode(data))
+	return string(utf16.Decode(data))
 }
 
 func uint32ToBytes(v uint32) []byte {
-       bytes := make([]byte, 4)
+	bytes := make([]byte, 4)
-       bytes[0] = byte(v & 0xff)
+	bytes[0] = byte(v & 0xff)
-       bytes[1] = byte((v >> 8) & 0xff)
+	bytes[1] = byte((v >> 8) & 0xff)
-       bytes[2] = byte((v >> 16) & 0xff)
+	bytes[2] = byte((v >> 16) & 0xff)
-       bytes[3] = byte((v >> 24) & 0xff)
+	bytes[3] = byte((v >> 24) & 0xff)
-       return bytes
+	return bytes
 }
-

ntlm/ntlmv1.go

@@ -6,7 +6,7 @@ import (
 	"bytes"
 	rc4P "crypto/rc4"
 	"errors"
-	l4g "github.com/ThomsonReutersEikon/log4go"
+	"log"
 	"strings"
 )
 
@@ -181,7 +181,7 @@ func (n *V1ServerSession) ProcessAuthenticateMessage(am *AuthenticateMessage) (e
 	// They should always be correct (I hope)
 	n.user = am.UserName.String()
 	n.userDomain = am.DomainName.String()
-	l4g.Info("(ProcessAuthenticateMessage)NTLM v1 User %s Domain %s", n.user, n.userDomain)
+	log.Printf("(ProcessAuthenticateMessage)NTLM v1 User %s Domain %s", n.user, n.userDomain)
 
 	err = n.fetchResponseKeys()
 	if err != nil {
@@ -225,7 +225,7 @@ func (n *V1ServerSession) ProcessAuthenticateMessage(am *AuthenticateMessage) (e
 		//UGH not entirely sure how this could possibly happen, going to put this in for now
 		//TODO investigate if this ever is really happening
 		am.Version = &VersionStruct{ProductMajorVersion: uint8(5), ProductMinorVersion: uint8(1), ProductBuild: uint16(2600), NTLMRevisionCurrent: uint8(15)}
-		l4g.Error("Nil version in ntlmv1")
+		log.Printf("Nil version in ntlmv1")
 	}
 
 	err = n.calculateKeys(am.Version.NTLMRevisionCurrent)

ntlm/ntlmv1_test.go

@@ -42,14 +42,14 @@ func checkV1Value(t *testing.T, name string, value []byte, expected string, err
 // would authenticate. This was due to a bug in the MS-NLMP docs. This tests for that issue
 func TestNtlmV1ExtendedSessionSecurity(t *testing.T) {
 	// NTLMv1 with extended session security
-  challengeMessage := "TlRMTVNTUAACAAAAAAAAADgAAABVgphiRy3oSZvn1I4AAAAAAAAAAKIAogA4AAAABQEoCgAAAA8CAA4AUgBFAFUAVABFAFIAUwABABwAVQBLAEIAUAAtAEMAQgBUAFIATQBGAEUAMAA2AAQAFgBSAGUAdQB0AGUAcgBzAC4AbgBlAHQAAwA0AHUAawBiAHAALQBjAGIAdAByAG0AZgBlADAANgAuAFIAZQB1AHQAZQByAHMALgBuAGUAdAAFABYAUgBlAHUAdABlAHIAcwAuAG4AZQB0AAAAAAA="
+	challengeMessage := "TlRMTVNTUAACAAAAAAAAADgAAABVgphiRy3oSZvn1I4AAAAAAAAAAKIAogA4AAAABQEoCgAAAA8CAA4AUgBFAFUAVABFAFIAUwABABwAVQBLAEIAUAAtAEMAQgBUAFIATQBGAEUAMAA2AAQAFgBSAGUAdQB0AGUAcgBzAC4AbgBlAHQAAwA0AHUAawBiAHAALQBjAGIAdAByAG0AZgBlADAANgAuAFIAZQB1AHQAZQByAHMALgBuAGUAdAAFABYAUgBlAHUAdABlAHIAcwAuAG4AZQB0AAAAAAA="
-  authenticateMessage := "TlRMTVNTUAADAAAAGAAYAJgAAAAYABgAsAAAAAAAAABIAAAAOgA6AEgAAAAWABYAggAAABAAEADIAAAAVYKYYgUCzg4AAAAPMQAwADAAMAAwADEALgB3AGMAcABAAHQAaABvAG0AcwBvAG4AcgBlAHUAdABlAHIAcwAuAGMAbwBtAE4AWQBDAFMATQBTAEcAOQA5ADAAOQBRWAK3h/TIywAAAAAAAAAAAAAAAAAAAAA3tp89kZU1hs1XZp7KTyGm3XsFAT9stEDW9YXDaeYVBmBcBb//2FOu"
+	authenticateMessage := "TlRMTVNTUAADAAAAGAAYAJgAAAAYABgAsAAAAAAAAABIAAAAOgA6AEgAAAAWABYAggAAABAAEADIAAAAVYKYYgUCzg4AAAAPMQAwADAAMAAwADEALgB3AGMAcABAAHQAaABvAG0AcwBvAG4AcgBlAHUAdABlAHIAcwAuAGMAbwBtAE4AWQBDAFMATQBTAEcAOQA5ADAAOQBRWAK3h/TIywAAAAAAAAAAAAAAAAAAAAA3tp89kZU1hs1XZp7KTyGm3XsFAT9stEDW9YXDaeYVBmBcBb//2FOu"
 
 	challengeData, _ := base64.StdEncoding.DecodeString(challengeMessage)
 	c, _ := ParseChallengeMessage(challengeData)
 
-  authenticateData, _ := base64.StdEncoding.DecodeString(authenticateMessage)
+	authenticateData, _ := base64.StdEncoding.DecodeString(authenticateMessage)
-  msg, err := ParseAuthenticateMessage(authenticateData, 1)
+	msg, err := ParseAuthenticateMessage(authenticateData, 1)
 	if err != nil {
 		t.Errorf("Could not process authenticate message: %s", err)
 	}

ntlm/ntlmv2.go

@@ -7,7 +7,7 @@ import (
 	rc4P "crypto/rc4"
 	"encoding/binary"
 	"errors"
-	l4g "github.com/ThomsonReutersEikon/log4go"
+	"log"
 	"strings"
 	"time"
 )
@@ -204,7 +204,7 @@ func (n *V2ServerSession) ProcessAuthenticateMessage(am *AuthenticateMessage) (e
 	// They should always be correct (I hope)
 	n.user = am.UserName.String()
 	n.userDomain = am.DomainName.String()
-	l4g.Info("(ProcessAuthenticateMessage)NTLM v2 User %s Domain %s", n.user, n.userDomain)
+	log.Printf("(ProcessAuthenticateMessage)NTLM v2 User %s Domain %s", n.user, n.userDomain)
 
 	err = n.fetchResponseKeys()
 	if err != nil {
@@ -238,14 +238,13 @@ func (n *V2ServerSession) ProcessAuthenticateMessage(am *AuthenticateMessage) (e
 		return err
 	}
 
-
 	if am.Version == nil {
-        //UGH not entirely sure how this could possibly happen, going to put this in for now
+		//UGH not entirely sure how this could possibly happen, going to put this in for now
-        //TODO investigate if this ever is really happening
+		//TODO investigate if this ever is really happening
-        am.Version = &VersionStruct{ProductMajorVersion: uint8(5), ProductMinorVersion: uint8(1), ProductBuild: uint16(2600), NTLMRevisionCurrent: uint8(15)}
+		am.Version = &VersionStruct{ProductMajorVersion: uint8(5), ProductMinorVersion: uint8(1), ProductBuild: uint16(2600), NTLMRevisionCurrent: uint8(15)}
 
-        l4g.Error("Nil version in ntlmv2")
+		log.Printf("Nil version in ntlmv2")
-    }
+	}
 
 	err = n.calculateKeys(am.Version.NTLMRevisionCurrent)
 	if err != nil {

utils/decode_auth.go

@@ -4,6 +4,8 @@ import (
 	"encoding/base64"
 	"flag"
 	"fmt"
+
+	"github.com/ThomsonReutersEikon/go-ntlm/ntlm"
 )
 
 func main() {

utils/test_auth.go

@@ -3,6 +3,7 @@ package main
 import (
 	"encoding/base64"
 	"fmt"
+
 	"github.com/ThomsonReutersEikon/go-ntlm/ntlm"
 )