Remove write lock option from the xsvm API

ava-labs · Oct 9, 2023 · 1ed1f59 · 1ed1f59
1 parent 6d53e51
commit 1ed1f59
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/vms/example/xsvm/api/server.go b/vms/example/xsvm/api/server.go
@@ -135,7 +135,10 @@ func (s *server) IssueTx(r *http.Request, args *IssueTxArgs, reply *IssueTxReply
 	}
 
 	ctx := r.Context()
-	if err := s.builder.AddTx(ctx, newTx); err != nil {
+	s.ctx.Lock.Lock()
+	err = s.builder.AddTx(ctx, newTx)
+	s.ctx.Lock.Unlock()
+	if err != nil {
 		return err
 	}
 
@@ -150,7 +153,9 @@ type LastAcceptedReply struct {
 }
 
 func (s *server) LastAccepted(_ *http.Request, _ *struct{}, reply *LastAcceptedReply) error {
+	s.ctx.Lock.RLock()
 	reply.BlockID = s.chain.LastAccepted()
+	s.ctx.Lock.RUnlock()
 	blkBytes, err := state.GetBlock(s.state, reply.BlockID)
 	if err != nil {
 		return err

diff --git a/vms/example/xsvm/vm.go b/vms/example/xsvm/vm.go
@@ -129,15 +129,12 @@ func (vm *VM) CreateHandlers(_ context.Context) (map[string]*common.HTTPHandler,
 		vm.chain,
 		vm.builder,
 	)
-	if err := server.RegisterService(api, Name); err != nil {
-		return nil, err
-	}
 	return map[string]*common.HTTPHandler{
 		"": {
-			LockOptions: common.WriteLock,
+			LockOptions: common.NoLock,
 			Handler:     server,
 		},
-	}, nil
+	}, server.RegisterService(api, Name)
 }
 
 func (*VM) HealthCheck(context.Context) (interface{}, error) {