Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

3.2.0 - dtag detection on send flow: more requirements #122

Closed
WietseWind opened this issue Dec 6, 2024 · 1 comment
Closed

3.2.0 - dtag detection on send flow: more requirements #122

WietseWind opened this issue Dec 6, 2024 · 1 comment
Labels
Implemented Prio Important
Milestone
3.3

Comments

@WietseWind
Copy link
Member

WietseWind commented Dec 6, 2024 •
edited
Loading

Important

Now if a certain of % of incoming txs have dtag we assume dtag during manual send flow

Drop spammers use tags too, causing false positives.

We should add to the requirements that the detection is only if asset is XRP and XRP drops > 1000 to be included in the dtag force.
@WietseWind WietseWind added the Prio Important label Jan 7, 2025
@WietseWind WietseWind added this to the 3.3 milestone Jan 8, 2025
@N3TC4T
Copy link
Member

N3TC4T commented Jan 17, 2025

Fixed in 99b291b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Implemented Prio Important
Projects
None yet
Milestone
3.3
Development

No branches or pull requests
2 participants
@WietseWind @N3TC4T