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.

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

Move black holes to future additions #242

Closed
wants to merge 1 commit into from
Closed

Move black holes to future additions #242

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Aug 11, 2014

No description provided.

@@ -1072,6 +1071,10 @@ When an escrow fund is unhealthy, lowering the aggression factor makes the escro
Escrow funds should generally be tuned to act slowly. This will allow arbitrage traders to do the heavy lifting, as the knowledge that the escrow fund will eventually get the price back to the target makes for a self-fulfilling prophecy when traders act on that knowledge. If the escrow fund acts too quickly, it loses money when the bitcoin version of a security leads the real-world version, as would happen if someone was engaging in insider trading anonymously using the bitcoin version.


### Black holes approach to versioning
All Master Protocol implementations are expected to keep pace with changes of this nature, but in the event one falls behind, it must treat addresses which broadcast messages using version numbers it does not recognize as "black holes". That is, any funds or properties which enter the control of that address are considered lost and unspendable, since that address is using a newer version of the Master Protocol. In the event that the out-dated implementation is upgraded to recognize the new message formats, the blockchain can be re-parsed, and nothing will be lost.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure, if you just wanted to move the block with a slightly different wording or if this is a proposal for a solution. If it's the later: this still results in a global blackout.

  1. You don't know what you don't know. Basically this includes any transaction that is not a valid Mastercoin transaction.
  2. Unrelated to the first issue: hiding incoming tokens is not sufficient. The prime example here is DEx activity. Say there is a sell intent transaction which your wallet identifies correctly, but a higher versioned accept/buy transaction which it doesn't identify. The buyer's purchase is zero'd anyway, given the higher versiond transaction, but what about the seller? Maybe the whole amount was bought, maybe not. Thus you'd need to blackout the seller, too.

@dexX7
Copy link
Member

dexX7 commented Jul 2, 2019

Hi,

in an attempt to clean up the current specification, I'm going to close old and outstanding pull requests. Please note, it's tagged as "old idea", so the work is not wasted and we can potentially review it at some point later.

Please feel free to resubmit it as new pull request, if you still think it's a good idea.

@dexX7 dexX7 closed this Jul 2, 2019
@dexX7 dexX7 added the old idea label Jul 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants