Skip to content

Add apple as an AuthProvider#70

Open
woat wants to merge 2 commits into
MystenLabs:mainfrom
woat:woat--add-apple-auth-url
Open

Add apple as an AuthProvider#70
woat wants to merge 2 commits into
MystenLabs:mainfrom
woat:woat--add-apple-auth-url

Conversation

@woat
Copy link
Copy Markdown

@woat woat commented Feb 17, 2025

Description

I'm adding apple based on https://docs.sui.io/guides/developer/cryptography/zklogin-integration#get-jwt

Test plan

I just ran this:

      const authUrl = await flow.createAuthorizationURL({
        provider: 'apple',
        network: 'mainnet',
        clientId: 'CLIENT_ID',
        redirectUrl: 'REDIRECT_URI',
      });

@woat woat requested a review from a team as a code owner February 17, 2025 22:59
@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 17, 2025

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

Name Status Preview Comments Updated (UTC)
sui-typescript-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 18, 2025 10:45pm

case 'apple': {
params.set('response_type', 'code%20id_token')
params.set('scope', 'email')
params.set('response_mode', 'form_post')
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I don’t think this response mode is compatible with most apps. We have plans to solve this eventually via a custom endpoint that can do the token exchange but haven’t built it out yet.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

image

So does apple auth work with Enoki at all? If so, is there some example flow I could look at?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It does, but it doesn’t work with the SDK. I’ll try and grab some resources around how to use it tomorrow and drop them here

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Ok thonks ser, I shall patiently wait for your response.
image

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

image

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