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

Split domain management #651

Closed
wants to merge 4 commits into from

Conversation

Anunayj
Copy link
Contributor

@Anunayj Anunayj commented Oct 26, 2021

This test should document how script addresses can be used to allow different entities to RENEW, UPDATE, and retain ownership of the domain without sharing keys.

This way watchtowers can be deployed whose sole purpose is to renew the domain, and have the ability to have a separate key for UPDATE(ing) DNS records on chain. While having a Cold Wallet (or Multisig) for the actual ownership/transfers.

HIP-0009 (handshake-org/HIPs#40) needs to be discussed to standardize how to derive the keys used for this.

This (branches from and) depends on PR #639 to be merged.

MTX clone() would not carry over coinview, in order to get around this the CoinSelector
would remove all inputs and add them back which made working with partially signed
transactions a pain, this should fix that issue.
Previous version of interactive-name-swap used a ugly hack to get around limitations
on wallet funding, since wallet.fund() no longer wipes previous inputs, the previous
implementation is not longer the recommended way.
@coveralls
Copy link

coveralls commented Oct 26, 2021

Pull Request Test Coverage Report for Build 1391888710

  • 24 of 25 (96.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.6%) to 62.26%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/primitives/mtx.js 11 12 91.67%
Totals Coverage Status
Change from base Build 1391687210: 0.6%
Covered Lines: 21187
Relevant Lines: 31813

💛 - Coveralls

@Anunayj Anunayj force-pushed the split-domain-management branch 2 times, most recently from 79bde70 to eecf36f Compare October 27, 2021 20:33
Example script to have seperate keys for UPDATEing and TRANSFERing domains + anyone can renew

Co-authored-by: Matthew Zipkin <pinheadmz@gmail.com>
@Anunayj Anunayj added tests part of the codebase docs part of the codebase needs conceptual review modifier waiting for other PR modifier and removed docs part of the codebase labels Feb 26, 2022
@Anunayj Anunayj closed this Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs conceptual review modifier tests part of the codebase waiting for other PR modifier
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants