-
Notifications
You must be signed in to change notification settings - Fork 249
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
chore: supporting missing auth config options #5011
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,6 +4,7 @@ | |
import 'package:amplify_core/amplify_core.dart'; | ||
import 'package:amplify_core/src/config/amplify_outputs/auth/identity_provider.dart'; | ||
import 'package:amplify_core/src/config/amplify_outputs/auth/oauth_response_type.dart'; | ||
import 'package:meta/meta.dart'; | ||
|
||
part 'oauth_outputs.g.dart'; | ||
|
||
|
@@ -19,7 +20,11 @@ class OAuthOutputs | |
required this.domain, | ||
required this.scopes, | ||
required this.redirectSignInUri, | ||
this.redirectSignInUriQueryParameters, | ||
required this.redirectSignOutUri, | ||
this.redirectSignOutUriQueryParameters, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. rename to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we alos need to add optional |
||
this.tokenUri, | ||
this.tokenUriQueryParameters, | ||
required this.responseType, | ||
}); | ||
|
||
|
@@ -38,9 +43,39 @@ class OAuthOutputs | |
/// URIs used to redirect after signing in using an identity provider. | ||
final List<String> redirectSignInUri; | ||
|
||
/// Optional query params in the sign in URI. | ||
/// | ||
/// Note: This attribute is not part of the Amplify Outputs schema. | ||
/// This exists for backwards compatibility with the Gen 1 config. | ||
@internal | ||
final Map<String, String>? redirectSignInUriQueryParameters; | ||
|
||
/// URIs used to redirect after signing out. | ||
final List<String> redirectSignOutUri; | ||
|
||
/// Optional query params in the sign in URI. | ||
/// | ||
/// Note: This attribute is not part of the Amplify Outputs schema. | ||
/// This exists for backwards compatibility with the Gen 1 config. | ||
@internal | ||
final Map<String, String>? redirectSignOutUriQueryParameters; | ||
|
||
/// The OAuth token URI. | ||
/// | ||
/// Defaults to '/oauth2/token'. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Q: Where is the default set? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In the auth plugin |
||
/// | ||
/// Note: This attribute is not part of the Amplify Outputs schema. | ||
/// This exists for backwards compatibility with the Gen 1 config. | ||
@internal | ||
final String? tokenUri; | ||
|
||
/// Optional query params in the OAuth token URI. | ||
/// | ||
/// Note: This attribute is not part of the Amplify Outputs schema. | ||
/// This exists for backwards compatibility with the Gen 1 config. | ||
@internal | ||
final Map<String, String>? tokenUriQueryParameters; | ||
|
||
/// {@macro amplify_core.amplify_outputs.oauth_response_type} | ||
final OAuthResponseType responseType; | ||
|
||
|
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.
rename to
signInUriQueryParameters
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.
we alos need to add optional
signInUri
fieldfinal String? signInUri;