Merge pull request #3117 from oasisprotocol/yawning/fix/trivial

go/oasis-node/cmd/common: ExportEntity should use the entity ctor
oasisprotocol · Jul 20, 2020 · b176b1f · b176b1f
2 parents 224d9cf + 4cea752
commit b176b1f
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/.changelog/3117.bugfix.md b/.changelog/3117.bugfix.md
@@ -0,0 +1,5 @@
+go/oasis-node/cmd/common: ExportEntity should use the entity ctor
+
+Instead of using an entity populated with the zero values and a manually
+filled in public key, use the entity constructor that can fill in
+sensible values for things like the version.
diff --git a/go/oasis-node/cmd/common/common.go b/go/oasis-node/cmd/common/common.go
@@ -272,7 +272,7 @@ func ExportEntity(signerBackend, entityDir string) error {
 	if err != nil {
 		return err
 	}
-	var entity entity.Entity
-	entity.ID = signer.Public()
-	return entity.Save(entityDir)
+
+	_, err = entity.GenerateWithSigner(entityDir, signer, nil)
+	return err
 }