-
Notifications
You must be signed in to change notification settings - Fork 79
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
feat!: World ID Bridge #127
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -0,0 +1,22 @@ | |||
export enum AppErrorCodes { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's make sure these error codes are exported for easy use
idkit/pnpm-lock.yaml
Outdated
@@ -1,4 +1,8 @@ | |||
lockfileVersion: '6.0' | |||
lockfileVersion: '6.1' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's remove WalletConnect deps
idkit/src/lib/crypto.ts
Outdated
const encoder = new TextEncoder() | ||
const decoder = new TextDecoder() | ||
|
||
export const generateKey = (): Promise<CryptoKeyPair> => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@recmo 👀
) | ||
} | ||
|
||
export const exportKey = async (key: CryptoKey): Promise<string> => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: make it explicit this is used to export the private key?
idkit/src/lib/crypto.ts
Outdated
export const exportKey = async (key: CryptoKey): Promise<string> => { | ||
const jwk = await window.crypto.subtle.exportKey('jwk', key) | ||
|
||
return Buffer.from(JSON.stringify(jwk)).toString('base64') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this the simplest encoding?
credential_types, | ||
action_description, | ||
action: encodeAction(action), | ||
external_nullifier: generateExternalNullifier(app_id, action).digest, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@m1guelpf we need to pass the external_nullifier to the app for the advanced on-chain cases and cases where devs aren't using the Dev Portal to register their apps/actions. however, I'm not sure if for the advanced on-chain use cases, this is the best way to do it, will devs rely on creating a "custom action" for this case?
Superseded by #156 |
No description provided.