chore_: remove private information for error log

abi-spec/utils.go

@@ -48,7 +48,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 {

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"
@@ -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
 			}
 
@@ -2367,7 +2368,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
 	}
 
@@ -2385,7 +2386,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
 	}
 
@@ -2399,7 +2400,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

@@ -120,7 +120,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))
@@ -242,7 +242,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
+	}
+
+	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"
@@ -478,13 +479,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))
@@ -658,7 +659,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))
@@ -767,7 +768,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))
@@ -794,7 +795,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))
@@ -2066,7 +2067,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
@@ -2099,6 +2100,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)
 }
@@ -2112,6 +2114,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)
 }
@@ -2159,6 +2162,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)
 }
@@ -2178,7 +2182,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
 }
@@ -2190,7 +2194,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)
@@ -2203,7 +2207,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)
@@ -2216,7 +2220,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
 }

node/geth_node.go

@@ -16,6 +16,7 @@ import (
 	"github.com/ethereum/go-ethereum/p2p/enode"
 	"github.com/ethereum/go-ethereum/p2p/nat"
 
+	"github.com/status-im/status-go/common"
 	"github.com/status-im/status-go/eth-node/crypto"
 	"github.com/status-im/status-go/logutils"
 	"github.com/status-im/status-go/params"
@@ -147,7 +148,7 @@ func parseNodes(enodes []string) []*enode.Node {
 		if err == nil {
 			nodes = append(nodes, parsedPeer)
 		} else {
-			logger.Error("Failed to parse enode", zap.String("enode", item), zap.Error(err))
+			logger.Error("Failed to parse enode", zap.String("enode", common.TruncateWithDot(item)), zap.Error(err))
 		}
 
 	}
@@ -163,7 +164,7 @@ func parseNodesV5(enodes []string) []*discv5.Node {
 		if err == nil {
 			nodes = append(nodes, parsedPeer)
 		} else {
-			logger.Error("Failed to parse enode", zap.String("enode", enode), zap.Error(err))
+			logger.Error("Failed to parse enode", zap.String("enode", common.TruncateWithDot(enode)), zap.Error(err))
 		}
 	}
 	return nodes

protocol/communities/manager.go

@@ -676,7 +676,7 @@ func (m *Manager) runOwnerVerificationLoop() {
 func (m *Manager) ValidateCommunityByID(communityID types.HexBytes) (*CommunityResponse, error) {
 	communitiesToValidate, err := m.persistence.getCommunityToValidateByID(communityID)
 	if err != nil {
-		m.logger.Error("failed to validate community by ID", zap.String("id", communityID.String()), zap.Error(err))
+		m.logger.Error("failed to validate community by ID", zap.String("id", utils.TruncateWithDot(communityID.String())), zap.Error(err))
 		return nil, err
 	}
 	return m.validateCommunity(communitiesToValidate)

protocol/messenger.go

@@ -2170,7 +2170,7 @@ func (m *Messenger) sendChatMessage(ctx context.Context, message *common.Message
 	if err == nil {
 		message.Text = replacedText
 	} else {
-		m.logger.Error("failed to replace text with public key", zap.String("chatID", message.ChatId), zap.String("text", message.Text))
+		m.logger.Error("failed to replace text with public key", zap.String("chatID", gocommon.TruncateWithDot(message.ChatId)), zap.Error(err))
 	}
 
 	if len(message.ImagePath) != 0 {