-
Notifications
You must be signed in to change notification settings - Fork 35
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
RIP-0013 accept change issuer #43
Comments
Probably a new field is needed Then original |
LGTM |
Hey, @Yuripetusko slid into my DMs a couple of days ago.
But having a May @Swader can explain the use case (why you need change issuer). In my thinking process, I only allow changing issuer for empty collections. |
|
No need for p |
|
Sorry not sure what happened there. And I don't remember what I was going to write. But I don't see a problem with pending Issuer approach |
Can you elaborate more on this @Swader ? If I'm paying for collection, should I explicitly accept it? Why is that? Isn't value in transaction enough?
How does this solve this ACCEPT? If the author (A) of the collection has compromised keys, how prevent ACCEPT not sending them away to desired and owned account?
Like if someone accidentally sends me their brand? |
The question was when is change issuer necessary. My answers address this question by giving a few examples, they do not focus on the accept side of things. |
RIP Summary
The new owner of the collection should fire an
ACCEPT
command after it was passed to him throughCHANGEISSUER
RIP Details
In order to prevent phishing attempts due to the fact that CHANGEISSUER only passes collection to the new owner and not NFTs, a new owner should ACCEPT incoming collection for issuer change to apply.
Original issuer retains all rights until the new issuer accepts it. The new issuer can only ACCEPT the incoming collection and nothing else
Examples
In the extrinsic version of RMRK standard, this can look like
RMRK::ACCEPTCOLLECTION::1.0.0::${collection_id}
Open Questions
Need to check a dump if someone did CHANGEISSUER multiple times on the same collection, as it should have failed because it wasn't accepted after the first time.
Prior work (optional)
?
Impact
In RMRK1 we need to make sure this doesn't affect any of existing collections. But since there's nothing that an issuer can do with his collection except for CHANGEISSUER again this shouldn't be a problem.
So the only thing that we need to check if someone did CHANGEISSUER multiple times on the same collection, as it should have failed because it wasn't accepted after the first time.
We also need to sync with Kodadot (@vikiival, @yangwao) and any indexer services like SubQuery and SubSquid
Thanks to @mmvds for hilighting this case
The text was updated successfully, but these errors were encountered: