fix: shift into platform_interface

firebase · MichaelVerdon · Jul 24, 2025 · Jun 19, 2025 · Jun 19, 2025 · Jun 20, 2025
commit 27f34768a1790d946e736f348500c0917486b7dd
diff --git a/packages/firebase_auth/firebase_auth/lib/firebase_auth.dart b/packages/firebase_auth/firebase_auth/lib/firebase_auth.dart
@@ -9,14 +9,6 @@ import 'package:firebase_core/firebase_core.dart';
 import 'package:firebase_core_platform_interface/firebase_core_platform_interface.dart';
 import 'package:flutter/foundation.dart';
 
-// Import internal password policy classes for implementation
-import 'src/internal.dart'
-    show
-        PasswordPolicy,
-        PasswordPolicyApi,
-        PasswordPolicyImpl,
-        PasswordValidationStatus;
-
 export 'package:firebase_auth_platform_interface/firebase_auth_platform_interface.dart'
     show
         FirebaseAuthException,
@@ -68,16 +60,14 @@ export 'package:firebase_auth_platform_interface/firebase_auth_platform_interfac
         RecaptchaVerifierOnExpired,
         RecaptchaVerifierOnError,
         RecaptchaVerifierSize,
-        RecaptchaVerifierTheme;
+        RecaptchaVerifierTheme,
+        PasswordPolicy,
+        PasswordPolicyApi,
+        PasswordPolicyImpl,
+        PasswordValidationStatus;
 export 'package:firebase_core_platform_interface/firebase_core_platform_interface.dart'
     show FirebaseException;
 
-export 'src/password_policy/password_validation_status.dart';
-// Remove internal class exports - these should not be part of public API
-// export 'src/password_policy/password_policy.dart';
-// export 'src/password_policy/password_policy_api.dart';
-// export 'src/password_policy/password_policy_impl.dart';
-
 part 'src/confirmation_result.dart';
 part 'src/firebase_auth.dart';
 part 'src/multi_factor.dart';

diff --git a/packages/firebase_auth/firebase_auth/lib/src/firebase_auth.dart b/packages/firebase_auth/firebase_auth/lib/src/firebase_auth.dart
@@ -858,23 +858,6 @@ class FirebaseAuth extends FirebasePluginPlatform {
   /// - **operation-not-allowed**:
   ///  - Thrown if email/password accounts are not enabled. Enable
   ///    email/password accounts in the Firebase Console, under the Auth tab.
-  Future<PasswordValidationStatus> validatePassword(
-    FirebaseAuth auth,
-    String? password,
-  ) async {
-    if (password == null || password.isEmpty) {
-      throw FirebaseAuthException(
-        code: 'invalid-password',
-        message: 'Password cannot be null or empty',
-      );
-    }
-    PasswordPolicyApi passwordPolicyApi = PasswordPolicyApi(auth);
-    PasswordPolicy passwordPolicy =
-        await passwordPolicyApi.fetchPasswordPolicy();
-    PasswordPolicyImpl passwordPolicyImpl = PasswordPolicyImpl(passwordPolicy);
-    return passwordPolicyImpl.isPasswordValid(password);
-  }
-
   @override
   String toString() {
     return 'FirebaseAuth(app: ${app.name})';

diff --git a/packages/firebase_auth/firebase_auth/lib/src/internal.dart b/packages/firebase_auth/firebase_auth/lib/src/internal.dart
diff --git a/packages/firebase_auth/firebase_auth/test/firebase_auth_test.dart b/packages/firebase_auth/firebase_auth/test/firebase_auth_test.dart
@@ -14,8 +14,6 @@ import 'package:flutter/foundation.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:mockito/mockito.dart';
 import 'package:plugin_platform_interface/plugin_platform_interface.dart';
-// Import internal password policy classes for testing
-import 'package:firebase_auth/src/internal.dart';
 
 import './mock.dart';
 

diff --git a/.../firebase_auth/firebase_auth_platform_interface/lib/firebase_auth_platform_interface.dart b/.../firebase_auth/firebase_auth_platform_interface/lib/firebase_auth_platform_interface.dart
@@ -39,3 +39,7 @@ export 'src/providers/play_games_auth.dart';
 export 'src/types.dart';
 export 'src/user_info.dart';
 export 'src/user_metadata.dart';
+export 'src/password_policy/password_policy_api.dart';
+export 'src/password_policy/password_policy_impl.dart';
+export 'src/password_policy/password_policy.dart';
+export 'src/password_policy/password_validation_status.dart';
diff --git a/.../src/password_policy/password_policy.dart → .../src/password_policy/password_policy.dart b/.../src/password_policy/password_policy.dart → .../src/password_policy/password_policy.dart
diff --git a/.../password_policy/password_policy_api.dart → .../password_policy/password_policy_api.dart b/.../password_policy/password_policy_api.dart → .../password_policy/password_policy_api.dart
@@ -2,24 +2,22 @@
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
-import 'package:firebase_auth/firebase_auth.dart';
 import 'package:http/http.dart' as http;
 import 'dart:convert';
 import 'dart:core';
 import 'password_policy.dart';
 
 class PasswordPolicyApi {
-  final FirebaseAuth _auth;
+  final String _apiKey;
   final String _apiUrl =
       'https://identitytoolkit.googleapis.com/v2/passwordPolicy?key=';
 
-  PasswordPolicyApi(this._auth);
+  PasswordPolicyApi(this._apiKey);
 
   final int _schemaVersion = 1;
 
   Future<PasswordPolicy> fetchPasswordPolicy() async {
     try {
-      final String _apiKey = _auth.app.options.apiKey;
       final response = await http.get(Uri.parse('$_apiUrl$_apiKey'));
       if (response.statusCode == 200) {
         final policy = json.decode(response.body);

diff --git a/...password_policy/password_policy_impl.dart → ...password_policy/password_policy_impl.dart b/...password_policy/password_policy_impl.dart → ...password_policy/password_policy_impl.dart
diff --git a/...rd_policy/password_validation_status.dart → ...rd_policy/password_validation_status.dart b/...rd_policy/password_validation_status.dart → ...rd_policy/password_validation_status.dart
diff --git a/packages/firebase_auth/firebase_auth_platform_interface/pubspec.yaml b/packages/firebase_auth/firebase_auth_platform_interface/pubspec.yaml
@@ -16,6 +16,7 @@ dependencies:
   firebase_core: ^3.15.1
   flutter:
     sdk: flutter
+  http: ^1.1.0
   meta: ^1.8.0
   plugin_platform_interface: ^2.1.3