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

Implemented PKCE Authorization Flow #65

Closed
wants to merge 6 commits into from

Conversation

ljubiknenad
Copy link

What kind of change does this PR introduce?

This PR consist implementation of PKCE Authorization Flow for getOAuthSignInURL & signInWithOTP methods.

What is the current behavior?

What is the new behavior?

When initializing a client there is a new property called flowType which is of type FlowType enum that consist of two types:

  • implicit
  • pkce

example:
let client = GoTrueClient(url: "", headers: [:], flowType: .pkce)

The flowType property has default value of .implicit

Additional context

Sources/GoTrue/GoTrueClient.swift Outdated Show resolved Hide resolved
Sources/GoTrue/Types.swift Outdated Show resolved Hide resolved
Sources/GoTrue/Endpoints.swift Outdated Show resolved Hide resolved
Sources/GoTrue/GoTrueClient.swift Show resolved Hide resolved
Sources/GoTrue/GoTrueError.swift Outdated Show resolved Hide resolved
Sources/GoTrue/GoTrueError.swift Outdated Show resolved Hide resolved
Sources/GoTrue/Types.swift Outdated Show resolved Hide resolved
Sources/GoTrue/Endpoints.swift Outdated Show resolved Hide resolved
Sources/GoTrue/Types.swift Outdated Show resolved Hide resolved
Sources/GoTrue/Types.swift Outdated Show resolved Hide resolved
@grdsdev grdsdev self-assigned this Oct 17, 2023
@grdsdev
Copy link
Contributor

grdsdev commented Oct 30, 2023

Closing in favor of: supabase/supabase-swift#119

@grdsdev grdsdev closed this Oct 30, 2023
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