8000 [Upstream] [Cleanup] Nuke obfuscation from orbit by lyricidal · Pull Request #241 · PRCYCoin/PRCYCoin · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[Upstream] [Cleanup] Nuke obfuscation from orbit #241

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
Merged
merged 17 commits into from
Aug 4, 2022
Merged

[Upstream] [Cleanup] Nuke obfuscation from orbit #241

merged 17 commits into from
Aug 4, 2022

Conversation

lyricidal
Copy link

"It's the only way to be sure!"

This PR removes the long-unused obfuscation classes, their related files, as well as all implementations throughout our codebase.

from PIVX-Project/PIVX#1490

Also includes a required commit: PIVX-Project/PIVX@171c56c (from PIVX-Project/PIVX#1024)

lyricidal added 17 commits August 4, 2022 14:02
@lyricidal
RPC: Remove getpoolinfo
9fa7864
@lyricidal
Move-only: Move ThreadCheckObfuScationPool to masternodeman
8b30daa
@lyricidal
Remove external obfuScationPool references
5720b0d
@lyricidal
Remove vecObfuscationQueue references
7453321
@lyricidal
Remove external mapObfuscationBroadcastTxes references
f9f8e4b
@lyricidal
Wallet: Remove obfuscation denomination references/code
8314681
@lyricidal
Remove CObfuScationRelay (obfuscation-relay.cpp/h) class and files
bcdff95
@lyricidal
Net: remove fObfuScationMaster from cNode
bd95953
@lyricidal
Remove obfuscation classes and files
1a773d9 
Also fixes up header includes due to the file removals
@lyricidal
Remove dstx network message type
fa2da87 
This message type was used for broadcasting obfuscation transactions
@lyricidal
[Refactor] Define messageSigner and hashSigner classes
db448a1
@lyricidal
Remove obfuscation transaction record types
f51f456
@lyricidal
Remove strObfuscationPoolDummyAddress param
82221db
@lyricidal
Remove obfuscation wording from output strings and code comments
1c22965
@lyricidal
Rename ThreadCheckObfuScationPool to ThreadCheckMasternodes
000947a
@lyricidal
RPC: Remove obfcompat field from getmasternodecount command
18dca35 
not needed, redundant with the `enabled` field.
@lyricidal
Remove nDsqCount from masternodeman's ToString function
9c4ae48 
Irrelevant data.
@lyricidal lyricidal added the Upstream Upstream backports/fixes label Aug 4, 2022
@lopeed lopeed merged commit fdf1b2b into develop Aug 4, 2022
@lopeed lopeed deleted the upstream-nuke_obfuscation-pr1490 branch August 4, 2022 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lopeed lopeed lopeed approved these changes

@anprdev anprdev anprdev approved these changes

@Shenkyy Shenkyy Awaiting requested review from Shenkyy

@lucablighthl lucablighthl Awaiting requested review from lucablighthl

@UserR256 UserR256 Awaiting requested review from UserR256

@Jamesj66 Jamesj66 Awaiting requested review from Jamesj66

@apophis1329 apophis1329 Awaiting requested review from apophis1329

Assignees
No one assigned
Labels
Upstream Upstream backports/fixes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lyricidal @lopeed @anprdev
0