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.

Sign up for GitHub

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

Jump to bottom

Refactor did:dht #68

Merged
merged 42 commits into from
Apr 25, 2024
Merged

Refactor did:dht #68

merged 42 commits into from
Apr 25, 2024

Conversation

mistermoe
Copy link
Member

@mistermoe mistermoe commented Apr 24, 2024 •
edited
Loading

Important

There's a decent amount of housekeeping that can be done still (variable naming etc.) but will tackle that in subsequent PRs. want to get this out the door!

TODO:

ethan-tbd and others added 28 commits April 23, 2024 23:58
@ethan-tbd @mistermoe
make DidDocument controller a list of strings
c0f1b83
@ethan-tbd @mistermoe
add create to Packet
ce477c2
@ethan-tbd @jiyoonie9 @mistermoe
fill out ServiceRecord
7afa4e4 
Co-Authored-By: Jiyoon Koo <118510103+jiyoontbd@users.noreply.github.com>
@ethan-tbd @mistermoe
add duplicate registered did type class
0ae8898
@ethan-tbd @mistermoe
add RootRecord
8e5b3a3
@ethan-tbd @mistermoe
fill out VerificationMethodRecord
835067e
@ethan-tbd @jiyoonie9 @mistermoe
add duplicate did dht and document packet bridge class
2f2b805 
Co-Authored-By: Jiyoon Koo <118510103+jiyoontbd@users.noreply.github.com>
Co-Authored-By: Moe Jangda <moe@tbd.email>
@mistermoe
temp bug fixes and add minimal bencoder
e19d3ef
@mistermoe
start work on bep44
12746a7
@mistermoe
add docs
f288b8b
@mistermoe
implement bep44 create and encode
12b5b4e
@ethan-tbd @mistermoe
remove RootRecord class
3c7d816
@ethan-tbd @mistermoe
add comments
1beacd3
@ethan-tbd @mistermoe
add root record and txt data maps into DocumentPacket
0b0cfeb
@ethan-tbd @mistermoe
update resolve() to use instance of DocumentPacket
074906f
@mistermoe
refactor DidDocument.addVerificationMethod to accept multiple purposes
d564cfa
@mistermoe
ditch old stuff and start refactoring packet creation loop
cfcef9b
@mistermoe
fix
d93c4d4
@mistermoe
add link to spec text
e5f64cc
@mistermoe
rename and set root record name
ac03e81
@mistermoe
implement VerificationMethodRecord.toVerificationMethod
a40b251
@mistermoe
moar
258512b
@mistermoe
refactor ServiceRecord.createService
5ec95f8
@mistermoe
add test
673cd73
@mistermoe
fix ServiceRecord.createTxtRecord
3210eb8
@mistermoe
add test for DocumentPacket.createDnsPacket
2b1a2d7
@mistermoe
cleanup
e234fce
@mistermoe
finish publishing logic
3be4ac0
This was linked to issues Apr 24, 2024
did:dht verify signature on resolution #12
Closed
Fix did:dht resolve #57
Closed
@mistermoe mistermoe marked this pull request as ready for review April 25, 2024 00:23
@mistermoe mistermoe merged commit 82176c7 into main Apr 25, 2024
1 check passed
@mistermoe mistermoe deleted the dht-refactor-step-2 branch April 25, 2024 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ethan-tbd ethan-tbd ethan-tbd approved these changes

@kirahsapong kirahsapong Awaiting requested review from kirahsapong

@wesbillman wesbillman Awaiting requested review from wesbillman wesbillman is a code owner

@diehuxx diehuxx Awaiting requested review from diehuxx

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix did:dht resolve did:dht verify signature on resolution
2 participants
@mistermoe @ethan-tbd