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

feat!: World ID Bridge #127

Closed
wants to merge 18 commits into from
Closed

feat!: World ID Bridge #127

wants to merge 18 commits into from

Conversation

m1guelpf
Copy link
Member

No description provided.

@m1guelpf m1guelpf self-assigned this May 24, 2023
@vercel
Copy link

vercel bot commented May 24, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
idkit-js-example ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 15, 2023 4:51am

@@ -0,0 +1,22 @@
export enum AppErrorCodes {
Copy link
Contributor

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

@@ -1,4 +1,8 @@
lockfileVersion: '6.0'
lockfileVersion: '6.1'
Copy link
Contributor

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

const encoder = new TextEncoder()
const decoder = new TextDecoder()

export const generateKey = (): Promise<CryptoKeyPair> => {
Copy link
Contributor

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> => {
Copy link
Contributor

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?

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')
Copy link
Contributor

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,
Copy link
Contributor

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?

@pdtfh pdtfh changed the title wip(🚧): bridge feat!: World ID Bridge Nov 9, 2023
@m1guelpf
Copy link
Member Author

Superseded by #156

@m1guelpf m1guelpf closed this Nov 15, 2023
@m1guelpf m1guelpf deleted the feat/bridge branch November 15, 2023 23:53
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.

4 participants