-
Notifications
You must be signed in to change notification settings - Fork 219
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
Remove parameters
from Authentication API client methods [SDK-2902]
#530
Conversation
parameters
param from Auth API client methods [SDK-2902]parameters
param from Authentication API client methods [SDK-2902]
parameters
param from Authentication API client methods [SDK-2902]parameters
from Authentication API client methods [SDK-2902]
48067b1
to
9beec35
Compare
@@ -1278,8 +1264,6 @@ public extension Authentication { | |||
.start { print($0) } | |||
``` | |||
|
|||
- precondition: if you are not using OAuth 2.0 API Authorization please use `delegation(parameters:)` |
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.
delegation(parameters:)
is deprecated and will be removed in a future PR.
7119c05
to
b740533
Compare
@@ -686,7 +678,7 @@ public protocol Authentication: Trackable, Loggable { | |||
|
|||
/** | |||
Renew user's credentials with a refresh_token grant for `/oauth/token` | |||
If you are not using OAuth 2.0 API Authorization please use `delegation(parameters:)` |
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.
delegation(parameters:)
is deprecated and will be removed in a future PR.
|
||
``` | ||
Auth0 | ||
.authentication(clientId: clientId, domain: "samples.auth0.com") | ||
.login(phoneNumber: "+4599134762367", | ||
code: "123456", | ||
audience: "https://myapi.com/api", | ||
scope: "openid email", |
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.
For a later PR, but make sure you have instructions somewhere for adding custom params to /authorize
Changes
The following Authentication API Client methods lost the
parameters
parameter:login(phoneNumber:code:audience:scope:)
login(usernameOrEmail:password:realm:audience:scope:)
loginDefaultDirectory(withUsername:password:audience:scope:)
tokenExchange()
That parameter was no longer necessary since
Request
has aparameters()
method that can be used to pass custom parameters to any Authentication API Client method.Testing
Checklist