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

[Draft] No passkey demo #49

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

[Draft] No passkey demo #49

wants to merge 6 commits into from

Conversation

cpb8010
Copy link
Contributor

@cpb8010 cpb8010 commented Jan 22, 2025

Description

We're looking to support owner keys that aren't passkey based in case the platform or user can't use passkeys
The contracts support this, but there are some small improvements that we can make to the SDK to better enable this.

Additional context

Depending on the use case (security vs ease of recovery), an embedded application may choose one of the 3 paths:

  1. Full EOA style unencrypted key export
  2. Export encrypted key with password and server managed salt (server could back up IV as recovery mechanism)
  3. Export initialization vector and have server manage encrypted key

This updates the interface so a passkey isn't required and samples creating a key
still need key export, but has an idea
still missing lots of UI parts, but not bad for some AI
Use mnemonic so it's a little easier to manage the non encrpyted version
@cpb8010 cpb8010 added the question Further information is requested label Jan 22, 2025
@cpb8010 cpb8010 self-assigned this Jan 22, 2025
Copy link

github-actions bot commented Jan 22, 2025

Visit the preview URL for this PR (updated for commit 1c3c383):

https://zksync-auth-server-staging--pr49-no-passkey-demo-sxqpl9a5.web.app

(expires Wed, 12 Feb 2025 03:23:34 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 509a9c9ea42583076f531c53cf2979c544d5d0b7

Still requires some server setup to provide the iv + salt
Still have test errors, probably related to parsing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant