-
Notifications
You must be signed in to change notification settings - Fork 84
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
SIP describing Stacks message signing #90
Comments
Is there a reason this method is preferred over using SIP018 with a Nonetheless, at Ryder have not implemented non-structured signing due to there not being a standard yet. If it were adopted as a SIP it be safer for devs to rely on the functionality. |
From my understanding from the last SIP call when this item was discussed Friedger and Jude were present, the conclusion on the I believe was to just for this to be part of SIP-018. Will let the others confirm this. |
Hi @kyranjamie @MarvinJanssen since SIP-018 will activate with SIP-015 (provided Stacks 2.1 vote will be a pass), do you guys need to make any amends to SIP-018 based on this above conversation? |
I made this issue to note that there's no SIP covering this functionality, that's currently supported by the Hiro Wallet and Ledger Stacks app. I'm no advocate for it, though. @MarvinJanssen's right that developers should favour SIP-018 going forward. Perhaps this functionality is better considered non-standard, not covered by a SIP, and eventually deprecated—that's up to Stacks app developers/community. I don't believe any action is necessary relating to upcoming SIP activations. |
Understood. Thank you Kyran for bringing this to people's awareness! |
@Hero-Gamer it also does not make sense to me to include it in SIP018 as the message signing described in the OP is quite different. |
No SIP exists describing how arbitrary messages are signed.
Specifics such as what the prefix is, and what encoding is used should be described.
EIP-712
, for example, has an arbitrary message section.The current prefix follows the same format as
EIP-712
, with Stacks in place of Ethereum.The need for this SIP comes from issues that've arisen migrating to a single prefix: See hirosystems/stacks.js#1328.
In migrating to a new prefix, some apps have seen breaking changes. Were this described in a SIP, a more structured, community-driven approach to changes would've been required.
cc/ @friedger @markmhx @MarvinJanssen @pradel @Hero-Gamer
The text was updated successfully, but these errors were encountered: