fix rollback and add logs

Author: Hitenjain14

code/go/0chain.net/blobbercore/handler/object_operation_handler.go

@@ -1047,6 +1047,7 @@ func (fsh *StorageHandler) CommitWriteV2(ctx context.Context, r *http.Request) (
 		ctx, writeMarker.AllocationRoot, clientKey, &numFiles, maxFiles, writeMarker.Timestamp, trie)
 	defer func() {
 		if !commitSuccess {
+			logging.Logger.Error("Rolling back the changes", zap.String("allocation_id", allocationID))
 			trie.Rollback()
 		}
 	}()
@@ -1073,10 +1074,12 @@ func (fsh *StorageHandler) CommitWriteV2(ctx context.Context, r *http.Request) (
 	}
 
 	if writemarkerEntity.WM.Size != size {
-		return nil, common.NewError("write_marker_validation_failed", fmt.Sprintf("Write Marker size %v does not match the connection size %v", writemarkerEntity.WM.Size, connectionObj.Size))
+		logging.Logger.Error("write_marker_validation_failed", zap.Any("write_marker_size", writemarkerEntity.WM.Size), zap.Any("connection_size", size), zap.String("allocation_id", allocationId))
+		return nil, common.NewError("write_marker_validation_failed", fmt.Sprintf("Write Marker size %v does not match the connection size %v", writemarkerEntity.WM.Size, size))
 	}
 
 	if latestWriteMarkerEntity != nil && latestWriteMarkerEntity.WM.ChainSize+size != writeMarker.ChainSize {
+		logging.Logger.Error("write_marker_validation_failed", zap.Any("write_marker_size", latestWriteMarkerEntity.WM.ChainSize+size), zap.Any("connection_size", writeMarker.ChainSize), zap.String("allocation_id", allocationId))
 		return nil, common.NewErrorf("invalid_chain_size",
 			"Invalid chain size. expected:%v got %v", latestWriteMarkerEntity.WM.ChainSize+connectionObj.Size, writeMarker.ChainSize)
 	} else if latestWriteMarkerEntity == nil && size != writeMarker.ChainSize {

code/go/0chain.net/blobbercore/handler/storage_handler.go

@@ -708,8 +708,8 @@ func (fsh *StorageHandler) getReferencePathV2(ctx context.Context, r *http.Reque
 		return
 	}
 	allocMu := lock.GetMutex(allocation.Allocation{}.TableName(), allocationId)
-	allocMu.Lock()
-	defer allocMu.Unlock()
+	allocMu.RLock()
+	defer allocMu.RUnlock()
 	now := time.Now()
 
 	trie := allocationObj.GetTrie()

go.mod

@@ -99,7 +99,7 @@ require (
 )
 
 require (
-	github.com/0chain/common v1.13.1-0.20240925093917-2680c8c6a0b0
+	github.com/0chain/common v1.13.1-0.20240925133253-58c8bc6998a6
 	github.com/Luzifer/go-openssl/v3 v3.1.0 // indirect
 	github.com/asaskevich/govalidator v0.0.0-20230301143203-a9d515a09cc2 // indirect
 	github.com/btcsuite/btcd v0.23.4 // indirect

go.sum

@@ -36,8 +36,8 @@ cloud.google.com/go/storage v1.8.0/go.mod h1:Wv1Oy7z6Yz3DshWRJFhqM/UCfaWIRTdp0RX
 cloud.google.com/go/storage v1.10.0/go.mod h1:FLPqc6j+Ki4BU591ie1oL6qBQGu2Bl/tZ9ullr3+Kg0=
 cloud.google.com/go/storage v1.14.0/go.mod h1:GrKmX003DSIwi9o29oFT7YDnHYwZoctc3fOKtUw0Xmo=
 dmitri.shuralyov.com/gpu/mtl v0.0.0-20190408044501-666a987793e9/go.mod h1:H6x//7gZCb22OMCxBHrMx7a5I7Hp++hsVxbQ4BYO7hU=
-github.com/0chain/common v1.13.1-0.20240925093917-2680c8c6a0b0 h1:WCHBXOjYtRWRwb5WtkWg94YTphMsi++HBE+EvdovxVE=
-github.com/0chain/common v1.13.1-0.20240925093917-2680c8c6a0b0/go.mod h1:vHK9Q0/nz28PxmKsLSMLzEs5iK6QVgl1LxdDQdR7y6s=
+github.com/0chain/common v1.13.1-0.20240925133253-58c8bc6998a6 h1:uYldZV/FDaZfT44IcNil4a4ipWS6afiiT4Efv22Jza8=
+github.com/0chain/common v1.13.1-0.20240925133253-58c8bc6998a6/go.mod h1:vHK9Q0/nz28PxmKsLSMLzEs5iK6QVgl1LxdDQdR7y6s=
 github.com/0chain/errors v1.0.3 h1:QQZPFxTfnMcRdt32DXbzRQIfGWmBsKoEdszKQDb0rRM=
 github.com/0chain/errors v1.0.3/go.mod h1:xymD6nVgrbgttWwkpSCfLLEJbFO6iHGQwk/yeSuYkIc=
 github.com/0chain/gosdk v1.17.0-RC6 h1:NB4pfu0VbVWfQQEfnFqn5inaiQuyhSaUW7VwgH3LW8U=