finish

main.go

@@ -9,6 +9,7 @@ import (
 	"github.com/solo-io/protoc-gen-ext/module/equal"
 	"github.com/solo-io/protoc-gen-ext/module/hash"
 	"github.com/solo-io/protoc-gen-ext/module/merge"
+	"github.com/solo-io/protoc-gen-ext/module/uniquehash"
 )
 
 func main() {
@@ -21,6 +22,7 @@ func main() {
 		equal.Equal(),
 		merge.Merge(),
 		clone.Clone(),
+		uniquehash.Hash(),
 	).RegisterPostProcessor(
 		pgsgo.GoFmt(),
 	).Render()

module/uniquehash/uniquehash.go

@@ -5,7 +5,7 @@ import (
 	pgsgo "github.com/lyft/protoc-gen-star/lang/go"
 	"github.com/solo-io/protoc-gen-ext/extproto"
 	"github.com/solo-io/protoc-gen-ext/templates"
-	"github.com/solo-io/protoc-gen-ext/templates/hash"
+	"github.com/solo-io/protoc-gen-ext/templates/uniquehash"
 )
 
 const (
@@ -28,7 +28,7 @@ func (m *HashModule) Name() string { return hasherName }
 
 func (m *HashModule) Execute(targets map[string]pgs.File, pkgs map[string]pgs.Package) []pgs.Artifact {
 	// Process file-level templates
-	tpl := templates.Template(m.Parameters(), hash.Register)
+	tpl := templates.Template(m.Parameters(), uniquehash.Register)
 
 	for _, f := range targets {
 		var hashAll bool

templates/hash/msg.go

@@ -2,6 +2,10 @@ package hash
 
 const msgTpl = `
 // Hash function
+//
+// Deprecated: due to hashing implemention only using field values. The omission
+// of the field name in the hash calculation can lead to hash collisions.
+// Prefer the HashUnique function instead.
 func (m {{ (msgTyp .).Pointer }}) Hash(hasher hash.Hash64) (uint64, error) {
 		if m == nil { return 0, nil }
 		if hasher == nil { hasher = fnv.New64() }