(#69) rename dependencies and remove need for choria/build

Author: ripienaar

glide.lock

@@ -12,6 +12,14 @@ const (
 	TransportV1     = "choria:transport:1"
 )
 
+// Secure controls the signing and validations of certificates in the protocol
+var Secure = "true"
+
+// IsSecure determines if this build will validate senders at protocol level
+func IsSecure() bool {
+	return Secure == "true"
+}
+
 // Additional to these the package for a specific version must also provide these constructors
 // with signature matching those in v1/constructors.go these are in use by mcollective/protocol.gos
 

protocol/protocol.go

@@ -6,8 +6,7 @@ import (
 	"fmt"
 	"time"
 
-	"github.com/choria-io/go-choria/build"
-	"github.com/choria-io/go-choria/protocol"
+	"github.com/choria-io/go-protocol/protocol"
 )
 
 // NewRequest creates a choria:request:1
@@ -167,7 +166,7 @@ func NewSecureReplyFromTransport(message protocol.TransportMessage) (secure prot
 func NewSecureRequest(request protocol.Request, publicCert string, privateCert string) (secure protocol.SecureRequest, err error) {
 	pubcerttxt := []byte("insecure")
 
-	if build.Secure == "true" {
+	if protocol.IsSecure() {
 		pubcerttxt, err = readFile(publicCert)
 		if err != nil {
 			err = fmt.Errorf("Could not read public certificate: %s", err.Error())

protocol/v1/constructors.go

@@ -3,7 +3,7 @@ package v1
 import (
 	"time"
 
-	"github.com/choria-io/go-choria/protocol"
+	"github.com/choria-io/go-protocol/protocol"
 	"github.com/tidwall/gjson"
 
 	. "github.com/onsi/ginkgo"

protocol/v1/reply_test.go

@@ -7,7 +7,7 @@ import (
 	"sync"
 	"time"
 
-	"github.com/choria-io/go-choria/protocol"
+	"github.com/choria-io/go-protocol/protocol"
 )
 
 type request struct {

protocol/v1/request.go

@@ -3,7 +3,7 @@ package v1
 import (
 	"time"
 
-	"github.com/choria-io/go-choria/protocol"
+	"github.com/choria-io/go-protocol/protocol"
 	. "github.com/onsi/ginkgo"
 	. "github.com/onsi/gomega"
 	"github.com/tidwall/gjson"

protocol/v1/request_test.go

@@ -8,7 +8,7 @@ import (
 	"strings"
 	"sync"
 
-	"github.com/choria-io/go-choria/protocol"
+	"github.com/choria-io/go-protocol/protocol"
 )
 
 // SecureReply contains 1 serialized Reply hashed

protocol/v1/security_reply.go

@@ -4,7 +4,7 @@ import (
 	"crypto/sha256"
 	"encoding/base64"
 
-	"github.com/choria-io/go-choria/protocol"
+	"github.com/choria-io/go-protocol/protocol"
 	"github.com/tidwall/gjson"
 
 	. "github.com/onsi/ginkgo"

protocol/v1/security_reply_test.go

@@ -18,8 +18,7 @@ import (
 	"strings"
 	"sync"
 
-	"github.com/choria-io/go-choria/build"
-	"github.com/choria-io/go-choria/protocol"
+	"github.com/choria-io/go-protocol/protocol"
 	log "github.com/sirupsen/logrus"
 )
 
@@ -55,7 +54,7 @@ func (r *secureRequest) SetMessage(request protocol.Request) (err error) {
 
 	r.Signature = "insecure"
 
-	if r.isSecure() {
+	if protocol.IsSecure() {
 		var signature []byte
 
 		signature, err = r.signString([]byte(j))
@@ -81,7 +80,7 @@ func (r *secureRequest) Valid() bool {
 	r.mu.Lock()
 	defer r.mu.Unlock()
 
-	if !r.isSecure() {
+	if !protocol.IsSecure() {
 		log.Debug("Bypassing validation on secure request due to build time flags")
 		return true
 	}
@@ -407,7 +406,3 @@ func readFile(path string) (cert []byte, err error) {
 
 	return
 }
-
-func (r *secureRequest) isSecure() bool {
-	return build.Secure == "true"
-}

protocol/v1/security_request.go

@@ -12,7 +12,7 @@ import (
 	"os"
 	"path/filepath"
 
-	"github.com/choria-io/go-choria/protocol"
+	"github.com/choria-io/go-protocol/protocol"
 	. "github.com/onsi/ginkgo"
 	. "github.com/onsi/gomega"
 	"github.com/sirupsen/logrus"

protocol/v1/security_request_test.go

@@ -7,7 +7,7 @@ import (
 	"strings"
 	"sync"
 
-	"github.com/choria-io/go-choria/protocol"
+	"github.com/choria-io/go-protocol/protocol"
 )
 
 type transportMessage struct {

protocol/v1/transport.go

@@ -1,7 +1,7 @@
 package v1
 
 import (
-	"github.com/choria-io/go-choria/protocol"
+	"github.com/choria-io/go-protocol/protocol"
 	. "github.com/onsi/ginkgo"
 	. "github.com/onsi/gomega"
 	"github.com/tidwall/gjson"