Browse Source

Don't override current pushnames in history sync

Tulir Asokan 3 years ago
parent
commit
5c1b57d7b8
3 changed files with 11 additions and 8 deletions
  1. 1 1
      go.mod
  2. 2 2
      go.sum
  3. 8 5
      historysync.go

+ 1 - 1
go.mod

@@ -8,7 +8,7 @@ require (
 	github.com/mattn/go-sqlite3 v1.14.9
 	github.com/prometheus/client_golang v1.11.0
 	github.com/skip2/go-qrcode v0.0.0-20200617195104-da1b6568686e
-	go.mau.fi/whatsmeow v0.0.0-20211109210528-7aea95b68ce3
+	go.mau.fi/whatsmeow v0.0.0-20211109215855-2e8804d7e690
 	golang.org/x/image v0.0.0-20210628002857-a66eb6448b8d
 	google.golang.org/protobuf v1.27.1
 	gopkg.in/yaml.v3 v3.0.0-20210107192922-496545a6307b

+ 2 - 2
go.sum

@@ -139,8 +139,8 @@ github.com/tidwall/sjson v1.2.3 h1:5+deguEhHSEjmuICXZ21uSSsXotWMA0orU783+Z7Cp8=
 github.com/tidwall/sjson v1.2.3/go.mod h1:5WdjKx3AQMvCJ4RG6/2UYT7dLrGvJUV1x4jdTAyGvZs=
 go.mau.fi/libsignal v0.0.0-20211109153248-a67163214910 h1:9FFhG0OmkuMau5UEaTgiUQ+7cSbtbOQ7hiWKdN8OI3I=
 go.mau.fi/libsignal v0.0.0-20211109153248-a67163214910/go.mod h1:AufGrvVh+00Nc07Jm4hTquh7yleZyn20tKJI2wCPAKg=
-go.mau.fi/whatsmeow v0.0.0-20211109210528-7aea95b68ce3 h1:UYFUEgnx2yfSLcsB753q3vyXi8vrN/VGCk/toIG20f0=
-go.mau.fi/whatsmeow v0.0.0-20211109210528-7aea95b68ce3/go.mod h1:8jUjOAi3xtGubxcZgG8uSHpAdyQXBRbWAfxkctX/4y4=
+go.mau.fi/whatsmeow v0.0.0-20211109215855-2e8804d7e690 h1:Yx5v2ut+kd5YG+bLhCxqPnCEZ+yau2wKsQerk3VYtbY=
+go.mau.fi/whatsmeow v0.0.0-20211109215855-2e8804d7e690/go.mod h1:8jUjOAi3xtGubxcZgG8uSHpAdyQXBRbWAfxkctX/4y4=
 golang.org/x/crypto v0.0.0-20170930174604-9419663f5a44/go.mod h1:6SG95UA2DQfeDnfUPMdvaQW0Q7yPrPDi9nlGo2tz2b4=
 golang.org/x/crypto v0.0.0-20180904163835-0709b304e793/go.mod h1:6SG95UA2DQfeDnfUPMdvaQW0Q7yPrPDi9nlGo2tz2b4=
 golang.org/x/crypto v0.0.0-20190308221718-c2843e01d9a2/go.mod h1:djNgcEr1/C05ACkg1iLfiJU5Ep61QUkGW8qpdssI0+w=

+ 8 - 5
historysync.go

@@ -394,11 +394,14 @@ func (portal *Portal) backfill(source *User, messages []*waProto.WebMessageInfo)
 			continue
 		}
 		if webMsg.GetPushName() != "" && webMsg.GetPushName() != "-" {
-			changed, _, err := source.Client.Store.Contacts.PutPushName(info.Sender, webMsg.GetPushName())
-			if err != nil {
-				source.log.Errorfln("Failed to save push name of %s from historical message in device store: %v", info.Sender, err)
-			} else if changed {
-				source.log.Debugfln("Got push name %s for %s from historical message", webMsg.GetPushName(), info.Sender)
+			existingContact, _ := source.Client.Store.Contacts.GetContact(info.Sender)
+			if !existingContact.Found || existingContact.PushName == "" {
+				changed, _, err := source.Client.Store.Contacts.PutPushName(info.Sender, webMsg.GetPushName())
+				if err != nil {
+					source.log.Errorfln("Failed to save push name of %s from historical message in device store: %v", info.Sender, err)
+				} else if changed {
+					source.log.Debugfln("Got push name %s for %s from historical message", webMsg.GetPushName(), info.Sender)
+				}
 			}
 		}
 		puppet := portal.getMessagePuppet(source, info)