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

New implementation based on Webnative 0.35 #12

Merged
merged 15 commits into from
Dec 15, 2022
Merged

Conversation

icidasset
Copy link
Contributor

Rewrote the Webnative integration from scratch. The result is a much smaller entry function and doesn't have the state problems like before.

The auth component has been replaced with crypto and manners components. Webnative signs UCANs using the crypto component's keystore functions, these connect to webnative-walletauth's wallet implementation. The manners component is used to store the root read-key encrypted on the user's file system and to retrieve + decrypt it as well. This is done by introducing file system hooks.

How to use, see README

Copy link
Member

@avivash avivash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking great! I'm aiming to finish up the WAT-react changes this morning then focus on the walletauth repos for the afternoon, so I'll go through and test everything more thoroughly then 👍🏼

@@ -1,220 +1,143 @@
import * as auth from "webnative/auth/index.js"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so much red in this file(and the rest of the PR), i love it! 🙌🏼

src/index.ts Outdated Show resolved Hide resolved
src/index.ts Outdated Show resolved Hide resolved
icidasset and others added 2 commits December 15, 2022 13:23
src/index.ts Show resolved Hide resolved
src/index.ts Outdated Show resolved Hide resolved
src/index.ts Show resolved Hide resolved
Copy link
Member

@avivash avivash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@icidasset icidasset merged commit dd4d1a5 into main Dec 15, 2022
@icidasset icidasset deleted the icidasset/webnative-0.35 branch December 15, 2022 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants