[#1689] getSvc: Stop node addresses iteratation on logical errors

Before this fix: If first node address returns non-logical error (context
deadline exceeded, context canceled, etc), then this error will override
logical error from second node address.

Change-Id: Ib30d0209828fdc83b55308ca2e33a361aa4caee6
Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>

pkg/services/object/get/v2/util.go

@@ -3,6 +3,7 @@ package getsvc
 import (
 	"context"
 	"crypto/sha256"
+	"errors"
 	"hash"
 
 	"git.frostfs.info/TrueCloudLab/frostfs-node/pkg/core/client"
@@ -358,19 +359,20 @@ func groupAddressRequestForwarder(f func(context.Context, network.Address, clien
 
 		info.AddressGroup().IterateAddresses(func(addr network.Address) (stop bool) {
 			var err error
-
-			defer func() {
-				stop = err == nil
-
-				if stop || firstErr == nil {
-					firstErr = err
-				}
-
-				// would be nice to log otherwise
-			}()
-
 			res, err = f(ctx, addr, c, key)
 
+			// non-status logic error that could be returned
+			// from the SDK client; should not be considered
+			// as a connection error
+			var siErr *objectSDK.SplitInfoError
+			var eiErr *objectSDK.ECInfoError
+
+			stop = err == nil || errors.As(err, &siErr) || errors.As(err, &eiErr)
+
+			if stop || firstErr == nil {
+				firstErr = err
+			}
+
 			return
 		})