studio - Firebase Android onAuthStateChanged() se dispara dos veces después de iniciar sesión con correo electrónico y contraseña()

no se puede iniciar sesion con firebase

Question

Cuando uso signInWithEmailAndPassword() para iniciar sesión en onAuthStateChanged() siempre se dispara dos veces.

Estoy seguro de que la escucha solo se agrega una vez a firebaseAuth, and I have the code in onStop () `para eliminar al oyente después de eso.

¿Alguien sabe como resolver esto?

Mi código:

public class SignInActivity extends BaseActivity implements
        View.OnClickListener,
        GoogleApiClient.OnConnectionFailedListener{

    private static final String PREF_KEY_USER_EMAIL = "User_Email";
    private static final int RC_SIGN_IN = 1111;

    private FirebaseAuth firebaseAuth;
    private FirebaseAuth.AuthStateListener authStateListener;
    private DatabaseReference firebaseDbReference;

    private TextView fieldEmail;
    private TextView fieldPassword;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_sign_in);
        getSupportActionBar().hide();

        firebaseAuth = FirebaseAuth.getInstance();
        firebaseDbReference = FirebaseDatabase.getInstance().getReference();
        fieldEmail = (TextView) findViewById(R.id.field_email);
        fieldPassword = (TextView) findViewById(R.id.field_password);

        String userSavedEmail = getPreferences(MODE_PRIVATE).getString(PREF_KEY_USER_EMAIL, "");
        if(userSavedEmail != null) {
            fieldEmail.setText(userSavedEmail);
            fieldPassword.requestFocus();
        }

        TextView linkForgotPassword;
        Button buttonLogin;

        linkForgotPassword = (TextView) findViewById(R.id.link_forgotPassword);
        buttonLogin = (Button) findViewById(R.id.button_Login);
        buttonSignUp = (Button) findViewById(R.id.button_signUp);

        if (linkForgotPassword != null) {
            linkForgotPassword.setOnClickListener(this);
        }
        if (buttonLogin != null) {
            buttonLogin.setOnClickListener(this);
        }
        if (buttonSignUp != null) {
            buttonSignUp.setOnClickListener(this);
        }

        authStateListener = new FirebaseAuth.AuthStateListener() {
            @Override
            public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {
                if(firebaseAuth.getCurrentUser() != null) {
                    onAuthSuccess(firebaseAuth.getCurrentUser());
                }
            }
        };
    }

    @Override
    protected void onStart() {
        super.onStart();
        firebaseAuth.addAuthStateListener(authStateListener);
    }

    @Override
    protected void onStop() {
        super.onStop();
        if (authStateListener != null) {
            firebaseAuth.removeAuthStateListener(authStateListener);
        }
    }

    @Override
    public void onClick(View v) {
        switch (v.getId()){
            case R.id.link_forgotPassword:
                forgotPassword();
                break;
            case R.id.button_Login:
                emailLogin();
                break;
            case R.id.button_signUp:
                emailSignUp();
                break;
        }
    }

    @Override
    public void onConnectionFailed(@NonNull ConnectionResult connectionResult) {
        Toast.makeText(this, "Google Play Services error.", Toast.LENGTH_SHORT).show();
    }

    private void forgotPassword(){
        FirebaseAuth auth = FirebaseAuth.getInstance();
        String emailAddress = fieldEmail.getText().toString();

        if(TextUtils.isEmpty(emailAddress)){
            Toast.makeText(SignInActivity.this, R.string.msg_EnterEmail,
                    Toast.LENGTH_SHORT).show();
        }
        else {
            showProgressDialog();
            auth.sendPasswordResetEmail(emailAddress)
                    .addOnCompleteListener(new OnCompleteListener<Void>() {
                        @Override
                        public void onComplete(@NonNull Task<Void> task) {
                            hideProgressDialog();
                            if (task.isSuccessful()) {
                                Toast.makeText(SignInActivity.this, R.string.msg_ResetPasswordEmailSent,
                                        Toast.LENGTH_LONG).show();
                            }
                        }
                    });
        }
    }

    private void emailLogin(){
        if (!validateForm()) {
            return;
        }

        showProgressDialog();
        String email = fieldEmail.getText().toString();
        String password = fieldPassword.getText().toString();

        firebaseAuth.signInWithEmailAndPassword(email, password)
                .addOnCompleteListener(new OnCompleteListener<AuthResult>() {
                    @Override
                    public void onComplete(@NonNull Task<AuthResult> task) {
                        hideProgressDialog();

                        if (!task.isSuccessful()) {
                            Toast.makeText(SignInActivity.this, R.string.msg_EmailLoginFailed,
                                    Toast.LENGTH_SHORT).show();
                        }
                        else {
                            // Save the email
                            getPreferences(MODE_PRIVATE).edit()
                                    .putString(PREF_KEY_USER_EMAIL, fieldEmail.getText().toString())
                                    .apply();
                        }
                    }
                });
    }

    private void emailSignUp(){
        if (!validateForm()) {
            return;
        }

        showProgressDialog();
        String email = fieldEmail.getText().toString();
        String password = fieldPassword.getText().toString();

        firebaseAuth.createUserWithEmailAndPassword(email, password)
                .addOnCompleteListener(new OnCompleteListener<AuthResult>() {
                    @Override
                    public void onComplete(@NonNull Task<AuthResult> task) {
                        hideProgressDialog();

                        if (task.isSuccessful()) {
                            FirebaseUser user = task.getResult().getUser();
                            String displayName = displayNameFromEmail(user.getEmail());

                            // Update profile display name.
                            UserProfileChangeRequest profileUpdates = new UserProfileChangeRequest.Builder()
                                    .setDisplayName("Jane Q. User")
                                    .build();
                            user.updateProfile(profileUpdates);
                        } else {
                            Toast.makeText(SignInActivity.this, R.string.msg_EmailSignUpFailed,
                                    Toast.LENGTH_SHORT).show();
                        }
                    }
                });
    }

    private void onAuthSuccess(FirebaseUser user) {
        // Write new user
        writeNewUser(user.getUid(),
                user.getDisplayName(),
                user.getEmail(),
                user.getPhotoUrl());

        // Go to MainActivity
        startActivity(new Intent(this.getApplicationContext(), MainActivity.class));
        finish();
    }

    private void writeNewUser(String userId, String displayName, String email, android.net.Uri photoUrl) {
        User user = new User(displayName, email);

        if(photoUrl != null){
            user.setPhotoUrl(photoUrl.toString());
        }

        firebaseDbReference.child("users").child(userId).setValue(user);
    }

    private String displayNameFromEmail(String email) {
        if (email.contains("@")) {
            return email.split("@")[0];
        } else {
            return email;
        }
    }

    private boolean validateForm() {
        boolean result = true;
        if (TextUtils.isEmpty(fieldEmail.getText().toString())) {
            fieldEmail.setError("Required");
            result = false;
        } else {
            fieldEmail.setError(null);
        }

        if (TextUtils.isEmpty(fieldPassword.getText().toString())) {
            fieldPassword.setError("Required");
            result = false;
        } else {
            fieldPassword.setError(null);
        }

        return result;
    }
}

Answer 1

La doble llamada se debe una llamada de registro. No solo eso, onAuthStateChanged se llamará muchas veces en muchos estados diferentes, sin posibilidad de saber qué estado es.

Documentation dice:

onAuthStateChanged (autenticación FirebaseAuth)

Este método se invoca en el subproceso de la interfaz de usuario en los cambios en el estado de autenticación:

• Justo después de que el oyente ha sido registrado

• Cuando un usuario ha iniciado sesión

• Cuando el usuario actual está desconectado
• Cuando el usuario actual cambia
• Cuando hay un cambio en el token del usuario actual

Aquí algunos consejos para descubrir el estado actual:

• Llamada de registro: salta la primera llamada con una bandera.
• El usuario ha iniciado sesión: el usuario del parámetro es! = Nulo.
• El usuario se desconectó: el usuario del parámetro es == nulo.
• Cambios actuales del usuario: ¡el usuario del parámetro es! = Nulo y la última identificación del usuario es! = El ID del usuario desde el parámetro
• Actualización del token de usuario: el usuario desde el parámetro es! = Nulo y la última identificación del usuario es == ID del usuario desde el parámetro

Este oyente es un desastre y muy bugprone. El equipo de Firebase debería investigarlo.

Answer 2

Lejos de ser ideal también, pero es una solución más precisa.

private String authFlag = "";

...

authStateListener = new FirebaseAuth.AuthStateListener() {
            @Override
            public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {    
            if (firebaseAuth.getCurrentUser() == null) {
              if (authFlag != null) {
                authFlag = null;
                subscriber.onNext(null);
              }
            } else {
              String uid = firebaseAuth.getCurrentUser().getUid();
              if (authFlag == null || authFlag.isEmpty() || !authFlag.equals(uid)) {
                authFlag = uid;
                subscriber.onNext(firebaseAuth.getCurrentUser());
              }
            }
}

Answer 3

Se dispara dos veces y creo que este es un error que los chicos de Firebase deberían solucionar (mirándote a ti, Frank jejeje). Lo único que puedo pensar que puedes hacer ahora es agregar una bandera como esta.

private boolean flag = true;
...
authStateListener = new FirebaseAuth.AuthStateListener() {
            @Override
            public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {
                if(firebaseAuth.getCurrentUser() != null && flag) {
                    onAuthSuccess(firebaseAuth.getCurrentUser());
                    flag=false;
                }
            }
        };

Lejos de ser ideal, pero funcionará por ahora. El código aún se dispara dos veces, aceptamos el primero y negamos el segundo con nuestra bandera, de esta forma, si los chicos de Firebase lo arreglan y de repente el oyente se ejecuta una vez, nuestro código aún funciona. Tal vez esté destinado a que el oyente corra dos veces, con suerte tendremos algunas respuestas de la publicación cruzada de Frank