[Cashtab] Flag msgs received from addresses not in contact list
ClosedPublic
Actions

Authored by bytesofman on Apr 21 2022, 22:27.

Details

Reviewers

emack
kieran709

Group Reviewers

Restricted Project

Commits

rABC9049081ac994: [Cashtab] Flag msgs received from addresses not in contact list

Summary

T2410, T2399

Flags incoming message transactions that originate from addresses not in the user's contact list

This is in response to the appearance of scam messages in Cashtab's messaging feature

Test Plan

npm start
Receive a Cashtab Message from an address that is in your contact list, observe it appears as normal
Receive a Cashtab Message from an address that is not in your contact list, observe a warning

Diff Detail

Repository

rABC Bitcoin ABC

Branch

contact-list-available-on-home

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 18862
Build 37510: Build Diff	cashtab-tests
Build 37509: arc lint + arc unit

Event Timeline

bytesofman created this revision.Apr 21 2022, 22:27

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 21 2022, 22:27

bytesofman requested review of this revision.Apr 21 2022, 22:27

Harbormaster completed remote builds in B18862: Diff 33278.Apr 21 2022, 22:30

emack requested changes to this revision.Apr 21 2022, 23:27

emack added inline comments.

web/cashtab/src/components/Home/Tx.js
598	There's a gap here whereby opreturn messages sent from ElectrumABC or any custom nodejs scripts will bypass this scam alert. since this block is already the case when data.opReturnMessage exists, suggest no need for the 'data.isCashtabMessage ?' condition.
web/cashtab/src/utils/validation.js
148	'contact' minor typo
167	&& contactObj.name.length < 24