chore_: remove private information for error log

abi-spec/utils.go

@@ -11,6 +11,7 @@ import (
 	"go.uber.org/zap"
 
 	"github.com/ethereum/go-ethereum/common"
+	gocommon "github.com/status-im/status-go/common"
 	"github.com/status-im/status-go/eth-node/crypto"
 	"github.com/status-im/status-go/logutils"
 )
@@ -48,7 +49,7 @@ func Sha3(str string) string {
 	if hexStrictPattern.MatchString(str) {
 		bytes, err = hex.DecodeString(str[2:])
 		if err != nil {
-			logutils.ZapLogger().Error("failed to decode hex string when sha3", zap.String("hex", str), zap.Error(err))
+			logutils.ZapLogger().Error("failed to decode hex string when sha3", zap.Error(err))
 			return ""
 		}
 	} else {
@@ -183,7 +184,7 @@ func ToChecksumAddress(address string) (string, error) {
 		return "", nil
 	}
 	if !addressBasicPattern.MatchString(address) {
-		return "", fmt.Errorf("given address '%s' is not a valid Ethereum address", address)
+		return "", fmt.Errorf("given address '%s' is not a valid Ethereum address", gocommon.TruncateWithDot(address))
 	}
 
 	address = strings.ToLower(address)

account/accounts.go

@@ -20,6 +20,7 @@ import (
 	gethcommon "github.com/ethereum/go-ethereum/common"
 	"github.com/ethereum/go-ethereum/common/hexutil"
 	"github.com/status-im/status-go/account/generator"
+	gocommon "github.com/status-im/status-go/common"
 	"github.com/status-im/status-go/eth-node/crypto"
 	"github.com/status-im/status-go/eth-node/keystore"
 	"github.com/status-im/status-go/eth-node/types"
@@ -230,7 +231,7 @@ func (m *DefaultManager) VerifyAccountPassword(keyStoreDir, address, password st
 
 	// avoid swap attack
 	if key.Address != addressObj {
-		return nil, fmt.Errorf("account mismatch: have %s, want %s", key.Address.Hex(), addressObj.Hex())
+		return nil, fmt.Errorf("account mismatch: have %s, want %s", gocommon.TruncateWithDot(key.Address.Hex()), gocommon.TruncateWithDot(addressObj.Hex()))
 	}
 
 	return key, nil

api/geth_backend.go

@@ -30,6 +30,7 @@ import (
 	"github.com/status-im/status-go/appdatabase"
 	"github.com/status-im/status-go/centralizedmetrics"
 	centralizedmetricscommon "github.com/status-im/status-go/centralizedmetrics/common"
+	gocommon "github.com/status-im/status-go/common"
 	"github.com/status-im/status-go/common/dbsetup"
 	"github.com/status-im/status-go/connection"
 	"github.com/status-im/status-go/eth-node/crypto"
@@ -282,7 +283,7 @@ func (b *GethStatusBackend) getAccountByKeyUID(keyUID string) (*multiaccounts.Ac
 			return &acc, nil
 		}
 	}
-	return nil, fmt.Errorf("account with keyUID %s not found", keyUID)
+	return nil, fmt.Errorf("account with keyUID %s not found", gocommon.TruncateWithDot(keyUID))
 }
 
 func (b *GethStatusBackend) SaveAccount(account multiaccounts.Account) error {
@@ -358,7 +359,7 @@ func (b *GethStatusBackend) DeleteImportedKey(address, password, keyStoreDir str
 		if strings.Contains(fileInfo.Name(), address) {
 			_, err := b.accountManager.VerifyAccountPassword(keyStoreDir, "0x"+address, password)
 			if err != nil {
-				b.logger.Error("failed to verify account", zap.String("account", address), zap.Error(err))
+				b.logger.Error("failed to verify account", zap.String("account", gocommon.TruncateWithDot(address)), zap.Error(err))
 				return err
 			}
 
@@ -2376,7 +2377,7 @@ func (b *GethStatusBackend) getVerifiedWalletAccount(address, password string) (
 	}
 	exists, err := db.AddressExists(types.HexToAddress(address))
 	if err != nil {
-		b.logger.Error("failed to query db for a given address", zap.String("address", address), zap.Error(err))
+		b.logger.Error("failed to query db for a given address", zap.String("address", gocommon.TruncateWithDot(address)), zap.Error(err))
 		return nil, err
 	}
 
@@ -2394,7 +2395,7 @@ func (b *GethStatusBackend) getVerifiedWalletAccount(address, password string) (
 	}
 
 	if err != nil {
-		b.logger.Error("failed to verify account", zap.String("account", address), zap.Error(err))
+		b.logger.Error("failed to verify account", zap.String("account", gocommon.TruncateWithDot(address)), zap.Error(err))
 		return nil, err
 	}
 
@@ -2408,7 +2409,7 @@ func (b *GethStatusBackend) generatePartialAccountKey(db *accounts.Database, add
 	dbPath, err := db.GetPath(types.HexToAddress(address))
 	path := "m/" + dbPath[strings.LastIndex(dbPath, "/")+1:]
 	if err != nil {
-		b.logger.Error("failed to get path for given account address", zap.String("account", address), zap.Error(err))
+		b.logger.Error("failed to get path for given account address", zap.String("account", gocommon.TruncateWithDot(address)), zap.Error(err))
 		return nil, err
 	}
 

appdatabase/database.go

@@ -128,7 +128,7 @@ func FixMissingKeyUIDForAccounts(sqlTx *sql.Tx) error {
 		}
 		pk, err := crypto.UnmarshalPubkey(pubkey)
 		if err != nil {
-			logutils.ZapLogger().Error("Migrating accounts: failed to unmarshal pubkey", zap.String("pubkey", string(pubkey)), zap.Error(err))
+			logutils.ZapLogger().Error("Migrating accounts: failed to unmarshal pubkey", zap.String("pubkey", d_common.TruncateWithDot(string(pubkey))), zap.Error(err))
 			return err
 		}
 		pkBytes := sha256.Sum256(crypto.FromECDSAPub(pk))
@@ -250,7 +250,7 @@ func migrateEnsUsernames(sqlTx *sql.Tx) error {
 
 		_, err = sqlTx.Exec(`INSERT INTO ens_usernames (username, chain_id) VALUES (?, ?)`, username, defaultChainID)
 		if err != nil {
-			logutils.ZapLogger().Error("Migrating ens usernames: failed to insert username into new database", zap.String("ensUsername", username), zap.Error(err))
+			logutils.ZapLogger().Error("Migrating ens usernames: failed to insert username into new database", zap.Error(err))
 		}
 	}
 

common/utils.go

@@ -22,6 +22,8 @@ var ErrInvalidDisplayNameNotAllowed = errors.New("name is not allowed")
 
 var DISPLAY_NAME_EXT = []string{"_eth", ".eth", "-eth"}
 
+const DefaultTruncateLength = 8
+
 func RecoverKey(m *protobuf.ApplicationMetadataMessage) (*ecdsa.PublicKey, error) {
 	if m.Signature == nil {
 		return nil, nil
@@ -103,3 +105,39 @@ func LogOnPanic() {
 
 	panic(err)
 }
+
+// TruncateWithDotN truncates a string by keeping n characters (excluding "..") from start and end,
+// replacing middle part with "..".
+// The total length of returned string will be n + 2 (where 2 is the length of "..").
+// For n characters total (excluding dots), characters are distributed evenly between start and end.
+// If n is odd, start gets one extra character.
+// Examples:
+//
+//	TruncateWithDotN("0x123456789", 4) // returns "0x1..89" (total length 6)
+//	TruncateWithDotN("0x123456789", 3) // returns "0x..9" (total length 5)
+//	TruncateWithDotN("0x123456789", 5) // returns "0x1..89" (total length 7)
+//	TruncateWithDotN("acdkef099", 3) // returns "ac..9" (total length 5)
+//	TruncateWithDotN("ab", 4)        // returns "ab" (no truncation needed as len <= n)
+//	TruncateWithDotN("test", 0)      // returns "test" (no truncation for n <= 0)
+func TruncateWithDotN(s string, n int) string {
+	if n <= 0 || len(s) <= n {
+		return s
+	}
+
+	if n < 2 { // need at least 2 chars (1 at start + 1 at end)
+		n = 2
+	}
+
+	const dots = ".."
+	// For n characters total (excluding dots), we want to keep:
+	// - ceil(n/2) characters from the start
+	// - floor(n/2) characters from the end
+	startChars := (n + 1) / 2 // ceil(n/2)
+	endChars := n / 2         // floor(n/2)
+
+	return s[:startChars] + dots + s[len(s)-endChars:]
+}
+
+func TruncateWithDot(s string) string {
+	return TruncateWithDotN(s, DefaultTruncateLength)
+}

common/utils_test.go

@@ -0,0 +1,97 @@
+package common
+
+import (
+	"testing"
+)
+
+func TestTruncateWithDotN(t *testing.T) {
+	tests := []struct {
+		name     string
+		input    string
+		n        int
+		expected string
+	}{
+		{
+			name:     "empty string",
+			input:    "",
+			n:        4,
+			expected: "",
+		},
+		{
+			name:     "string shorter than n",
+			input:    "123",
+			n:        4,
+			expected: "123",
+		},
+		{
+			name:     "string equal to n",
+			input:    "1234",
+			n:        4,
+			expected: "1234",
+		},
+		{
+			name:     "hex string with n=4 (even split: 2+2)",
+			input:    "0x123456789",
+			n:        4,
+			expected: "0x..89",
+		},
+		{
+			name:     "hex string with n=3 (odd split: 2+1)",
+			input:    "0x123456789",
+			n:        3,
+			expected: "0x..9",
+		},
+		{
+			name:     "hex string with n=5 (odd split: 3+2)",
+			input:    "0x123456789",
+			n:        5,
+			expected: "0x1..89",
+		},
+		{
+			name:     "normal string with n=3 (odd split: 2+1)",
+			input:    "acdkef099",
+			n:        3,
+			expected: "ac..9",
+		},
+		{
+			name:     "hex address with n=6 (even split: 3+3)",
+			input:    "0xd8dA6BF26964aF9D7eEd9e03E53415D37aA96045",
+			n:        6,
+			expected: "0xd..045",
+		},
+		{
+			name:     "very short n=2 (even split: 1+1)",
+			input:    "abcdef",
+			n:        2,
+			expected: "a..f",
+		},
+		{
+			name:     "n=1 adjusted to minimum 2 (even split: 1+1)",
+			input:    "abcdef",
+			n:        1,
+			expected: "a..f",
+		},
+		{
+			name:     "negative n",
+			input:    "test",
+			n:        -1,
+			expected: "test",
+		},
+		{
+			name:     "zero n",
+			input:    "test",
+			n:        0,
+			expected: "test",
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result := TruncateWithDotN(tt.input, tt.n)
+			if result != tt.expected {
+				t.Errorf("TruncateWithDotN(%q, %d) = %q, want %q",
+					tt.input, tt.n, result, tt.expected)
+			}
+		})
+	}
+}

eth-node/bridge/geth/ens/ens.go

@@ -68,13 +68,13 @@ func (m *Verifier) verifyENSName(ensInfo enstypes.ENSDetails, ethclient *ethclie
 	// Resolve ensName
 	resolver, err := ens.NewResolver(ethclient, ensName)
 	if err != nil {
-		m.logger.Error("error while creating ENS name resolver", zap.String("ensName", ensName), zap.Error(err))
+		m.logger.Error("error while creating ENS name resolver", zap.Error(err))
 		response.Error = err
 		return response
 	}
 	x, y, err := resolver.PubKey()
 	if err != nil {
-		m.logger.Error("error while resolving public key from ENS name", zap.String("ensName", ensName), zap.Error(err))
+		m.logger.Error("error while resolving public key from ENS name", zap.Error(err))
 		response.Error = err
 		return response
 	}

mobile/status.go

@@ -25,6 +25,7 @@ import (
 	"github.com/status-im/status-go/api/multiformat"
 	"github.com/status-im/status-go/centralizedmetrics"
 	"github.com/status-im/status-go/centralizedmetrics/providers"
+	gocommon "github.com/status-im/status-go/common"
 	"github.com/status-im/status-go/eth-node/crypto"
 	"github.com/status-im/status-go/eth-node/types"
 	"github.com/status-im/status-go/exportlogs"
@@ -474,13 +475,13 @@ func login(accountData, password, configJSON string) error {
 		logutils.ZapLogger().Debug("start a node with account", zap.String("key-uid", account.KeyUID))
 		err := statusBackend.UpdateNodeConfigFleet(account, password, &conf)
 		if err != nil {
-			logutils.ZapLogger().Error("failed to update node config fleet", zap.String("key-uid", account.KeyUID), zap.Error(err))
+			logutils.ZapLogger().Error("failed to update node config fleet", zap.String("key-uid", gocommon.TruncateWithDot(account.KeyUID)), zap.Error(err))
 			return statusBackend.LoggedIn(account.KeyUID, err)
 		}
 
 		err = statusBackend.StartNodeWithAccount(account, password, &conf, nil)
 		if err != nil {
-			logutils.ZapLogger().Error("failed to start a node", zap.String("key-uid", account.KeyUID), zap.Error(err))
+			logutils.ZapLogger().Error("failed to start a node", zap.String("key-uid", gocommon.TruncateWithDot(account.KeyUID)), zap.Error(err))
 			return err
 		}
 		logutils.ZapLogger().Debug("started a node with", zap.String("key-uid", account.KeyUID))
@@ -654,7 +655,7 @@ func SaveAccountAndLogin(accountData, password, settingsJSON, configJSON, subacc
 		logutils.ZapLogger().Debug("starting a node, and saving account with configuration", zap.String("key-uid", account.KeyUID))
 		err := statusBackend.StartNodeWithAccountAndInitialConfig(account, password, settings, &conf, subaccs, nil)
 		if err != nil {
-			logutils.ZapLogger().Error("failed to start node and save account", zap.String("key-uid", account.KeyUID), zap.Error(err))
+			logutils.ZapLogger().Error("failed to start node and save account", zap.String("key-uid", gocommon.TruncateWithDot(account.KeyUID)), zap.Error(err))
 			return err
 		}
 		logutils.ZapLogger().Debug("started a node, and saved account", zap.String("key-uid", account.KeyUID))
@@ -763,7 +764,7 @@ func SaveAccountAndLoginWithKeycard(accountData, password, settingsJSON, configJ
 		logutils.ZapLogger().Debug("starting a node, and saving account with configuration", zap.String("key-uid", account.KeyUID))
 		err := statusBackend.SaveAccountAndStartNodeWithKey(account, password, settings, &conf, subaccs, keyHex)
 		if err != nil {
-			logutils.ZapLogger().Error("failed to start node and save account", zap.String("key-uid", account.KeyUID), zap.Error(err))
+			logutils.ZapLogger().Error("failed to start node and save account", zap.String("key-uid", gocommon.TruncateWithDot(account.KeyUID)), zap.Error(err))
 			return err
 		}
 		logutils.ZapLogger().Debug("started a node, and saved account", zap.String("key-uid", account.KeyUID))
@@ -790,7 +791,7 @@ func LoginWithKeycard(accountData, password, keyHex string, configJSON string) s
 		logutils.ZapLogger().Debug("start a node with account", zap.String("key-uid", account.KeyUID))
 		err := statusBackend.StartNodeWithKey(account, password, keyHex, &conf)
 		if err != nil {
-			logutils.ZapLogger().Error("failed to start a node", zap.String("key-uid", account.KeyUID), zap.Error(err))
+			logutils.ZapLogger().Error("failed to start a node", zap.String("key-uid", gocommon.TruncateWithDot(account.KeyUID)), zap.Error(err))
 			return err
 		}
 		logutils.ZapLogger().Debug("started a node with", zap.String("key-uid", account.KeyUID))
@@ -2062,7 +2063,7 @@ func EncodeTransfer(to string, value string) string {
 func encodeTransfer(to string, value string) string {
 	result, err := abi_spec.EncodeTransfer(to, value)
 	if err != nil {
-		logutils.ZapLogger().Error("failed to encode transfer", zap.String("to", to), zap.String("value", value), zap.Error(err))
+		logutils.ZapLogger().Error("failed to encode transfer", zap.String("to", gocommon.TruncateWithDot(to)), zap.String("value", gocommon.TruncateWithDot(value)), zap.Error(err))
 		return ""
 	}
 	return result
@@ -2095,6 +2096,7 @@ func encodeFunctionCall(method string, paramsJSON string) string {
 	return result
 }
 
+// Deprecated: no usage in mobile, we might implemented it within other functions
 func EncodeFunctionCallV2(requestJSON string) string {
 	return callWithResponse(encodeFunctionCallV2, requestJSON)
 }
@@ -2108,6 +2110,7 @@ func encodeFunctionCallV2(requestJSON string) string {
 	return encodeFunctionCall(request.Method, request.ParamsJSON)
 }
 
+// Deprecated: no usage in mobile
 func DecodeParameters(decodeParamJSON string) string {
 	return decodeParameters(decodeParamJSON)
 }
@@ -2155,6 +2158,7 @@ func Sha3(str string) string {
 	return "0x" + abi_spec.Sha3(str)
 }
 
+// Deprecated: no usage in mobile
 func Utf8ToHex(str string) string {
 	return callWithResponse(utf8ToHex, str)
 }
@@ -2174,7 +2178,7 @@ func HexToUtf8(hexString string) string {
 func hexToUtf8(hexString string) string {
 	str, err := abi_spec.HexToUtf8(hexString)
 	if err != nil {
-		logutils.ZapLogger().Error("failed to convert hex to utf8", zap.String("hexString", hexString), zap.Error(err))
+		logutils.ZapLogger().Error("failed to convert hex to utf8", zap.String("hexString", gocommon.TruncateWithDot(hexString)), zap.Error(err))
 	}
 	return str
 }
@@ -2186,7 +2190,7 @@ func CheckAddressChecksum(address string) string {
 func checkAddressChecksum(address string) string {
 	valid, err := abi_spec.CheckAddressChecksum(address)
 	if err != nil {
-		logutils.ZapLogger().Error("failed to invoke check address checksum", zap.String("address", address), zap.Error(err))
+		logutils.ZapLogger().Error("failed to invoke check address checksum", zap.String("address", gocommon.TruncateWithDot(address)), zap.Error(err))
 	}
 	result, _ := json.Marshal(valid)
 	return string(result)
@@ -2199,7 +2203,7 @@ func IsAddress(address string) string {
 func isAddress(address string) string {
 	valid, err := abi_spec.IsAddress(address)
 	if err != nil {
-		logutils.ZapLogger().Error("failed to invoke IsAddress", zap.String("address", address), zap.Error(err))
+		logutils.ZapLogger().Error("failed to invoke IsAddress", zap.String("address", gocommon.TruncateWithDot(address)), zap.Error(err))
 	}
 	result, _ := json.Marshal(valid)
 	return string(result)
@@ -2212,7 +2216,7 @@ func ToChecksumAddress(address string) string {
 func toChecksumAddress(address string) string {
 	address, err := abi_spec.ToChecksumAddress(address)
 	if err != nil {
-		logutils.ZapLogger().Error("failed to convert to checksum address", zap.String("address", address), zap.Error(err))
+		logutils.ZapLogger().Error("failed to convert to checksum address", zap.String("address", gocommon.TruncateWithDot(address)), zap.Error(err))
 	}
 	return address
 }