Fix data race on service Proposal

Signed-off-by: Anton Litvinov <[email protected]>
mysteriumnetwork · Aug 30, 2024 · 64aba04 · 64aba04
1 parent 1f879f5
commit 64aba04
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/tequilapi/endpoints/service.go b/tequilapi/endpoints/service.go
@@ -346,6 +346,12 @@ func (se *ServiceEndpoint) toServiceOptions(serviceType string, value *json.RawM
 }
 
 func (se *ServiceEndpoint) toServiceInfoResponse(id service.ID, instance *service.Instance) (contract.ServiceInfoDTO, error) {
+
+	// Warning.
+	// Use only safely obtained copy of instance.Proposal field, otherwise concurrent
+	// access to that field from Instance.proposalWithCurrentLocation
+	// can trigger a data race
+
 	priced, err := se.proposalRepository.EnrichProposalWithPrice(instance.CopyProposal())
 	if err != nil {
 		return contract.ServiceInfoDTO{}, err