Удаление данных из базы данных RealTime - Firebase - PullRequest
0 голосов
/ 17 сентября 2018

Я использую базу данных Firebase в реальном времени, и для нее настроены следующие правила,

{
  "rules": {
    ".read": "auth !== null",
    ".write": "auth !== null"
  }
}

Но когда я пытаюсь удалить запись, выдает ошибку, в которой отказано в разрешении следующим образом.

Database: Client doesn't have permission to access the desired data. (database/permission-denied).

Что мне делать? Я не могу понять, почему я могу делать все чтения и записи довольно хорошо с текущими правилами и не удалять.

Может ли кто-нибудь помочь мне с этим?

Примечание: Изображение результатов, связанных с Firebase Simulator

enter image description here

При проверке на firebase.auth().currentUser:

deleteUserAccount(userId) {
 let knownLocationRef = this.database.ref('lastKnown/' + userId);
 let promises = [];
 console.log('auth details', this.auth.currentUser);
 console.log('auth details null', this.auth.currentUser !== null); //returns true
         knownLocationRef.once('value').then( (data) => {
            console.log('data ', data.val());
            if (data.val() !== null) {
                let prevLat = data.val().lat;
                let prevLong = data.val().long;
                console.log('auth details', this.auth.currentUser);
                console.log('auth details null', this.auth.currentUser !== null); //returns false
                promises.push(knownLocationRef.remove());
            }
        });
         return Promise.all(promises);
 }

1 Ответ

0 голосов
/ 17 сентября 2018

Вот некоторые проблемы с вашим кодом и с тем, как вы обрабатываете обещания:

function deleteUserAccount(userId) {
  let knownLocationRef = this.database.ref('lastKnown/' + userId);
  let promises = []; // good idea for collecting a list of distinct 
                     // promises, maybe not here

  // This is a promise, 'then' is a continuation from which you can return 
  // either a data result or another promise
  knownLocationRef.once('value').then((data) => {
    // if data is valid, return another promise
    if (data.val() !== null) {
      // ...

      // If you were looping and collecting promises, then this might help, 
      // but you don't want to collect inner promises.
      promises.push(knownLocationRef.remove());
    }
  });

  // This returns before everything in `.then` executes
  return Promise.all(promises);
}

Вот исправленная версия:

function deleteUserAccount(userId) {
  let knownLocationRef = this.database.ref('lastKnown/' + userId);
  let promises = []; // For the sake of argument, I will use this to collect 
                     // outer promises.

  // normally I would return here, but as an example, I'll uses the 
  // promises array
  promises.push(knownLocationRef.once('value').then((data) => {
    // if data is valid, return another promise
    if (data.val() !== null) {
      // ...

      // Return a Promise from a promise
      return knownLocationRef.remove();
    }

    return null; // no data to process
  }));

  // wait on outer promises
  return Promise.all(promises)
    .catch(console.log); // log errors from any promise
}

Вот версия без дополнительного promises массива.

function deleteUserAccount(userId) {
  let knownLocationRef = this.database.ref('lastKnown/' + userId);

  return knownLocationRef.once('value')
    .then((data) => {
      // if data is valid, return another promise
      if (data.val() !== null) {

        // Return a Promise from a promise
        return knownLocationRef.remove();
      }

      return null; // no data to process
    })
    .catch(console.log);
}
...