TF-4049 Pre-fill OIDC login form

dab246 · dab246 · commit f88085108be9 · 2025-10-08T14:50:45.000+07:00
diff --git a/lib/features/email/presentation/utils/email_utils.dart b/lib/features/email/presentation/utils/email_utils.dart
@@ -1,6 +1,7 @@
 import 'package:core/presentation/state/failure.dart';
 import 'package:core/presentation/state/success.dart';
 import 'package:core/utils/app_logger.dart';
+import 'package:core/utils/mail/domain.dart';
 import 'package:core/utils/mail/mail_address.dart';
 import 'package:get/get_utils/src/get_utils/get_utils.dart';
 import 'package:dartz/dartz.dart';
@@ -274,4 +275,24 @@ class EmailUtils {
       return '';
     }
   }
+
+  static MailAddress? getMailAddress({required String ownerEmail}) {
+    try {
+      return MailAddress.validateAddress(ownerEmail);
+    } catch (e) {
+      if (GetUtils.isEmail(ownerEmail)) {
+        final listPart = ownerEmail.split('@');
+        if (listPart.length == 2) {
+          return MailAddress(
+            localPart: listPart.first,
+            domain: Domain.of((listPart.last)),
+          );
+        }
+      }
+      return null;
+    }
+  }
+
+  static String? getLocalPartEmail(String emailAddress) =>
+      getMailAddress(ownerEmail: emailAddress)?.localPart;
 }
diff --git a/lib/features/login/data/datasource/authentication_oidc_datasource.dart b/lib/features/login/data/datasource/authentication_oidc_datasource.dart
@@ -8,7 +8,13 @@ abstract class AuthenticationOIDCDataSource {
 
   Future<OIDCDiscoveryResponse> discoverOIDC(OIDCConfiguration oidcConfiguration);
 
-  Future<TokenOIDC> getTokenOIDC(String clientId, String redirectUrl, String discoveryUrl, List<String> scopes);
+  Future<TokenOIDC> getTokenOIDC(
+    String clientId,
+    String redirectUrl,
+    String discoveryUrl,
+    List<String> scopes, {
+    String? loginHint,
+  });
 
   Future<void> persistTokenOIDC(TokenOIDC tokenOidc);
 
diff --git a/lib/features/login/data/datasource_impl/authentication_oidc_datasource_impl.dart b/lib/features/login/data/datasource_impl/authentication_oidc_datasource_impl.dart
@@ -56,9 +56,21 @@ class AuthenticationOIDCDataSourceImpl extends AuthenticationOIDCDataSource {
   }
 
   @override
-  Future<TokenOIDC> getTokenOIDC(String clientId, String redirectUrl, String discoveryUrl, List<String> scopes) {
+  Future<TokenOIDC> getTokenOIDC(
+    String clientId,
+    String redirectUrl,
+    String discoveryUrl,
+    List<String> scopes, {
+    String? loginHint,
+  }) {
     return Future.sync(() async {
-      return await _authenticationClient.getTokenOIDC(clientId, redirectUrl, discoveryUrl, scopes);
+      return await _authenticationClient.getTokenOIDC(
+        clientId,
+        redirectUrl,
+        discoveryUrl,
+        scopes,
+        loginHint: loginHint,
+      );
     }).catchError(_exceptionThrower.throwException);
   }
 
diff --git a/lib/features/login/data/network/authentication_client/authentication_client_base.dart b/lib/features/login/data/network/authentication_client/authentication_client_base.dart
@@ -13,10 +13,12 @@ abstract class AuthenticationClientBase {
       List<String> scopes);
 
   Future<TokenOIDC> getTokenOIDC(
-      String clientId,
-      String redirectUrl,
-      String discoveryUrl,
-      List<String> scopes);
+    String clientId,
+    String redirectUrl,
+    String discoveryUrl,
+    List<String> scopes, {
+    String? loginHint,
+  });
 
   Future<TokenOIDC> refreshingTokensOIDC(
       String clientId,
diff --git a/lib/features/login/data/network/authentication_client/authentication_client_interaction_mixin.dart b/lib/features/login/data/network/authentication_client/authentication_client_interaction_mixin.dart
@@ -59,14 +59,16 @@ mixin AuthenticationClientInteractionMixin {
     String clientId,
     String redirectUrl,
     String discoveryUrl,
-    List<String> scopes,
-  ) {
+    List<String> scopes, {
+    String? loginHint,
+  }) {
     return AuthorizationTokenRequest(
       clientId,
       redirectUrl,
       discoveryUrl: discoveryUrl,
       scopes: scopes,
       externalUserAgent: getExternalUserAgent(),
+      loginHint: loginHint,
     );
   }
 
diff --git a/lib/features/login/data/network/authentication_client/authentication_client_mobile.dart b/lib/features/login/data/network/authentication_client/authentication_client_mobile.dart
@@ -27,13 +27,15 @@ class AuthenticationClientMobile with AuthenticationClientInteractionMixin
     String clientId,
     String redirectUrl,
     String discoveryUrl,
-    List<String> scopes,
-  ) async {
+    List<String> scopes, {
+    String? loginHint,
+  }) async {
     final authorizationTokenRequest = getAuthorizationTokenRequest(
       clientId,
       redirectUrl,
       discoveryUrl,
       scopes,
+      loginHint: loginHint,
     );
     final authorizationTokenResponse = await _appAuth.authorizeAndExchangeCode(
       authorizationTokenRequest,
diff --git a/lib/features/login/data/network/authentication_client/authentication_client_web.dart b/lib/features/login/data/network/authentication_client/authentication_client_web.dart
@@ -25,13 +25,15 @@ class AuthenticationClientWeb with AuthenticationClientInteractionMixin
     String clientId,
     String redirectUrl,
     String discoveryUrl,
-    List<String> scopes,
-  ) async {
+    List<String> scopes, {
+    String? loginHint,
+  }) async {
     final authorizationTokenRequest = getAuthorizationTokenRequest(
       clientId,
       redirectUrl,
       discoveryUrl,
       scopes,
+      loginHint: loginHint,
     );
     final authorizationTokenResponse = await _appAuthWeb.authorizeAndExchangeCode(
       authorizationTokenRequest,
diff --git a/lib/features/login/data/repository/authentication_oidc_repository_impl.dart b/lib/features/login/data/repository/authentication_oidc_repository_impl.dart
@@ -28,8 +28,20 @@ class AuthenticationOIDCRepositoryImpl extends AuthenticationOIDCRepository {
   }
 
   @override
-  Future<TokenOIDC> getTokenOIDC(String clientId, String redirectUrl, String discoveryUrl, List<String> scopes) {
-    return _oidcDataSource.getTokenOIDC(clientId, redirectUrl, discoveryUrl, scopes);
+  Future<TokenOIDC> getTokenOIDC(
+    String clientId,
+    String redirectUrl,
+    String discoveryUrl,
+    List<String> scopes, {
+    String? loginHint,
+  }) {
+    return _oidcDataSource.getTokenOIDC(
+      clientId,
+      redirectUrl,
+      discoveryUrl,
+      scopes,
+      loginHint: loginHint,
+    );
   }
 
   @override
diff --git a/lib/features/login/domain/repository/authentication_oidc_repository.dart b/lib/features/login/domain/repository/authentication_oidc_repository.dart
@@ -8,7 +8,13 @@ abstract class AuthenticationOIDCRepository {
 
   Future<OIDCDiscoveryResponse> discoverOIDC(OIDCConfiguration oidcConfiguration);
 
-  Future<TokenOIDC> getTokenOIDC(String clientId, String redirectUrl, String discoveryUrl, List<String> scopes);
+  Future<TokenOIDC> getTokenOIDC(
+    String clientId,
+    String redirectUrl,
+    String discoveryUrl,
+    List<String> scopes, {
+    String? loginHint,
+  });
 
   Future<void> persistTokenOIDC(TokenOIDC tokenOidc);
 
diff --git a/lib/features/login/domain/usecases/get_oidc_configuration_interactor.dart b/lib/features/login/domain/usecases/get_oidc_configuration_interactor.dart
@@ -2,6 +2,7 @@ import 'package:core/presentation/state/failure.dart';
 import 'package:core/presentation/state/success.dart';
 import 'package:core/utils/app_logger.dart';
 import 'package:dartz/dartz.dart';
+import 'package:model/oidc/oidc_configuration.dart';
 import 'package:model/oidc/response/oidc_response.dart';
 import 'package:tmail_ui_user/features/login/domain/model/base_url_oidc_response.dart';
 import 'package:tmail_ui_user/features/login/domain/repository/authentication_oidc_repository.dart';
@@ -12,12 +13,20 @@ class GetOIDCConfigurationInteractor {
 
   GetOIDCConfigurationInteractor(this._oidcRepository);
 
-  Stream<Either<Failure, Success>> execute(OIDCResponse oidcResponse) async* {
+  Stream<Either<Failure, Success>> execute(
+    OIDCResponse oidcResponse, {
+    String? loginHint,
+  }) async* {
     try {
       yield Right<Failure, Success>(GetOIDCConfigurationLoading());
       final oidcConfiguration = await _oidcRepository.getOIDCConfiguration(oidcResponse);
-      await _oidcRepository.persistOidcConfiguration(oidcConfiguration);
-      yield Right<Failure, Success>(GetOIDCConfigurationSuccess(oidcConfiguration));
+      final configWithLoginHint = oidcConfiguration.copyWidth(
+        loginHint: loginHint,
+      );
+      await _oidcRepository.persistOidcConfiguration(configWithLoginHint);
+      yield Right<Failure, Success>(
+        GetOIDCConfigurationSuccess(configWithLoginHint),
+      );
     } catch (e) {
       logError('$runtimeType::execute():oidcResponse = ${oidcResponse.runtimeType} | Exception = $e');
       if (oidcResponse is BaseUrlOidcResponse) {
diff --git a/lib/features/login/domain/usecases/get_token_oidc_interactor.dart b/lib/features/login/domain/usecases/get_token_oidc_interactor.dart
@@ -27,10 +27,12 @@ class GetTokenOIDCInteractor {
     try {
       yield Right<Failure, Success>(GetTokenOIDCLoading());
       final tokenOIDC = await authenticationOIDCRepository.getTokenOIDC(
-          config.clientId,
-          config.redirectUrl,
-          config.discoveryUrl,
-          config.scopes);
+        config.clientId,
+        config.redirectUrl,
+        config.discoveryUrl,
+        config.scopes,
+        loginHint: config.loginHint,
+      );
 
       await Future.wait([
         _credentialRepository.saveBaseUrl(baseUri),
diff --git a/lib/features/login/presentation/login_controller.dart b/lib/features/login/presentation/login_controller.dart
@@ -18,6 +18,7 @@ import 'package:model/oidc/oidc_configuration.dart';
 import 'package:model/oidc/request/oidc_request.dart';
 import 'package:model/oidc/response/oidc_response.dart';
 import 'package:tmail_ui_user/features/base/reloadable/reloadable_controller.dart';
+import 'package:tmail_ui_user/features/email/presentation/utils/email_utils.dart';
 import 'package:tmail_ui_user/features/home/domain/state/auto_sign_in_via_deep_link_state.dart';
 import 'package:tmail_ui_user/features/home/domain/state/get_session_state.dart';
 import 'package:tmail_ui_user/features/login/data/network/oidc_error.dart';
@@ -188,7 +189,7 @@ class LoginController extends ReloadableController {
     } else if (success is CheckOIDCIsAvailableSuccess) {
       getOIDCConfiguration(success.oidcResponse);
     } else if (success is GetOIDCConfigurationSuccess) {
-      _getOIDCConfigurationSuccess(success);
+      _getOIDCConfigurationSuccess(success.oidcConfiguration);
     } else if (success is GetTokenOIDCSuccess) {
       _getTokenOIDCSuccess(success);
     } else if (success is AuthenticationUserSuccess) {
@@ -413,16 +414,25 @@ class LoginController extends ReloadableController {
   }
 
   void getOIDCConfiguration(OIDCResponse oidcResponse) {
-    consumeState(_getOIDCConfigurationInteractor.execute(oidcResponse));
+    final loginHint = PlatformInfo.isMobile && _username != null
+        ? EmailUtils.getLocalPartEmail(_username!.value) ?? _username?.value
+        : null;
+    log('$runtimeType::getOIDCConfiguration:loginHint = $loginHint');
+    consumeState(
+      _getOIDCConfigurationInteractor.execute(
+        oidcResponse,
+        loginHint: loginHint,
+      ),
+    );
   }
 
-  void _getOIDCConfigurationSuccess(GetOIDCConfigurationSuccess success) {
+  void _getOIDCConfigurationSuccess(OIDCConfiguration config) {
     if (PlatformInfo.isWeb) {
-      _authenticateOidcOnBrowserAction(success.oidcConfiguration);
-    } else if (success.oidcConfiguration.authority == AppConfig.saasRegistrationUrl) {
-      _getTokenOIDCOnSaaSPlatform(success.oidcConfiguration);
+      _authenticateOidcOnBrowserAction(config);
+    } else if (config.authority == AppConfig.saasRegistrationUrl) {
+      _getTokenOIDCOnSaaSPlatform(config);
     } else {
-      _getTokenOIDCAction(success.oidcConfiguration);
+      _getTokenOIDCAction(config);
     }
   }
 
diff --git a/lib/features/paywall/domain/model/paywall_url_pattern.dart b/lib/features/paywall/domain/model/paywall_url_pattern.dart
@@ -1,7 +1,5 @@
-import 'package:core/utils/mail/domain.dart';
-import 'package:core/utils/mail/mail_address.dart';
 import 'package:equatable/equatable.dart';
-import 'package:get/get.dart';
+import 'package:tmail_ui_user/features/email/presentation/utils/email_utils.dart';
 import 'package:tmail_ui_user/features/paywall/presentation/paywall_utils.dart';
 
 class PaywallUrlPattern with EquatableMixin {
@@ -10,31 +8,14 @@ class PaywallUrlPattern with EquatableMixin {
   PaywallUrlPattern(this.pattern);
 
   String getQualifiedUrl({required String ownerEmail, String? domainName}) {
-    final mailAddress = _getMailAddress(ownerEmail: ownerEmail);
+    final mailAddress = EmailUtils.getMailAddress(ownerEmail: ownerEmail);
     return PaywallUtils.buildPaywallUrlFromTemplate(
       template: pattern,
       localPart: mailAddress?.localPart,
       domainName: domainName ?? mailAddress?.domain.domainName,
     );
   }
 
-  MailAddress? _getMailAddress({required String ownerEmail}) {
-    try {
-      return MailAddress.validateAddress(ownerEmail);
-    } catch (e) {
-      if (GetUtils.isEmail(ownerEmail)) {
-        final listPart = ownerEmail.split('@');
-        if (listPart.length == 2) {
-          return MailAddress(
-            localPart: listPart.first,
-            domain: Domain.of((listPart.last)),
-          );
-        }
-      }
-      return null;
-    }
-  }
-
   @override
   List<Object> get props => [pattern];
 }
diff --git a/model/lib/oidc/oidc_configuration.dart b/model/lib/oidc/oidc_configuration.dart
@@ -8,12 +8,14 @@ class OIDCConfiguration with EquatableMixin {
   final String clientId;
   final List<String> scopes;
   final bool isTWP;
+  final String? loginHint;
 
   OIDCConfiguration({
     required this.authority,
     required this.clientId,
     required this.scopes,
     this.isTWP = false,
+    this.loginHint,
   });
 
   String get discoveryUrl {
@@ -30,5 +32,23 @@ class OIDCConfiguration with EquatableMixin {
     clientId,
     scopes,
     isTWP,
+    loginHint,
   ];
 }
+
+extension OIDCConfigurationExtension on OIDCConfiguration {
+  OIDCConfiguration copyWidth({
+    String? authority,
+    String? clientId,
+    List<String>? scopes,
+    bool? isTWP,
+    String? loginHint,
+  }) =>
+      OIDCConfiguration(
+        authority: authority ?? this.authority,
+        clientId: clientId ?? this.clientId,
+        scopes: scopes ?? this.scopes,
+        isTWP: isTWP ?? this.isTWP,
+        loginHint: loginHint ?? this.loginHint,
+      );
+}
