-
Notifications
You must be signed in to change notification settings - Fork 318
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
CIP-0070? | Add rejected CIP for spending policies #336
CIP-0070? | Add rejected CIP for spending policies #336
Conversation
This adds a rejected CIP recording a possible design that has been discussed in the past for "spending policies". I wanted to have somewhere to point when it comes up, and also to publicly record the reasons why designs like this are difficult.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understand this proposal at a high level but would want @KtorZ and/or @SebastienGllmt to agree that it should be put on record with these details.
@michaelpj I also think it's appropriate to have a Rejected
status and have argued for this in #331 (comment)... expecting your earlier conversation there will be resolved in favour of adding the Rejected
status back into the CIP workflow 😎
Yeah, this can have whatever the "not gonna happen" status is, when you decide on one :) |
Authors: Michael Peyton Jones <[email protected]> | ||
Status: Rejected | ||
Type: Standards Track | ||
Created: 23/09/2022 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Created: 23/09/2022 | |
Created: 2022-09-23 |
ISO-8601 please 🙏
This should probably actually become a CPS. |
I've recently discussed that this could be implemented in a way that I don't believe was discussed before -- that we can modify the event that triggers a spending policy to only happen when we move tokens between two different spending addresses (no invocation on any other change). I believe that would be sufficient to create tokens that
This would also allow us to safely use them in wallets as at any time the user may isolate the token to a 'garbage' utxo. |
|
On: transfer of token from a locking/spending address A to a spending address B. A change of stake key and change of datum (and any other utxo-carried information) does not trigger the Spending policy to run under this definition.
Only in this context. Choosing to not act IMO should be done inside the script.
Yes, but that case is now largely silo'd out into outgoing transfers which may now be isolated and understood on a case by case basis. The feature is a burden for wallets but now we can build an off-chain knowledge base to be able to construct transactions in a way that this spending policy would find acceptable. |
Flagging this as "Waiting for author" from @michaelpj's earlier comment:
Seems like a good idea to me indeed; we've taken quite some time processing that proposal and although we agreed with the general idea, CPS didn't exist at the time and could now be a better fit indeed. I am fine either way: merging as an inactive CIP or turning this to a CPS. |
Note @zygomeb @fallen-icarus (as you have shown some interested) that this indeed should have been a CPS... unfortunately we were not fortunate enough, as the CPS was evolving, to have the benefit of Michael's understanding in rewriting this as a CPS. We have to close this now because editors don't have the skills to rephrase it as a CPS, but someone in the community might take that over: likely as a new PR since the authorship would be different. Editors are standing by to cross-reference the old & new discussions if that ever happens. cc @Ryun1 @Crypto2099 |
This adds a rejected CIP recording a possible design that has been discussed in the past for "spending policies". I wanted to have somewhere to point when it comes up, and also to publicly record the reasons why designs like this are difficult.
see rendered Markdown