Axios / Vue - предотвращение выполнения axios.all () - PullRequest
0 голосов
/ 20 февраля 2019

В моем приложении, аутентифицирующем пользователя, я вызываю функцию fetchData.Если токен пользователя станет недействительным, приложение запустится axios.all() и мой перехватчик выдаст много ошибок.

Как предотвратить продолжение работы axios.all() после первой ошибки?И показывать только одно уведомление пользователю?

interceptors.js

export default (http, store, router) => {
    http.interceptors.response.use(response => response, (error) => {
        const {response} = error;

        let message = 'Ops. Algo de errado aconteceu...';

        if([401].indexOf(response.status) > -1){
            localforage.removeItem('token');

            router.push({
                name: 'login'
            });

            Vue.notify({
                group: 'panel',
                type: 'error',
                duration: 5000,
                text: response.data.message ? response.data.message : message
            });
        }

        return Promise.reject(error);
    })
}

auth.js

const actions = {
    fetchData({commit, dispatch}) {
        function getChannels() {
            return http.get('channels')
        }

        function getContacts() {
            return http.get('conversations')
        }

        function getEventActions() {
            return http.get('events/actions')
        }

        // 20 more functions calls

        axios.all([
            getChannels(),
            getContacts(),
            getEventActions()
        ]).then(axios.spread(function (channels, contacts, eventActions) {
            dispatch('channels/setChannels', channels.data, {root: true})
            dispatch('contacts/setContacts', contacts.data, {root: true})
            dispatch('events/setActions', eventActions.data, {root: true})
        }))
    }
}

Ответы [ 3 ]

0 голосов
/ 26 марта 2019

Ответ upvoted предлагает решение, которое требует ожидания всех ответов для завершения, зависимости от uuid и некоторой сложности в вашем перехватчике.Мое решение избегает всего этого и направлено на вашу цель прекращения Promise.all() выполнения.

Axios поддерживает отмену запросов , поэтому вы можете обернуть ваши GET запросы обработчиком ошибок, который отменяет другиенемедленно ожидающие запросы:

fetchData({ dispatch }) {
  const source = axios.CancelToken.source();

  // wrapper for GET requests
  function get(url) {
    return axios.get(url, {
        cancelToken: source.token // watch token for cancellation
      }).catch(error => {
        if (axios.isCancel(error)) {
          console.warn(`canceled ${url}, error: ${error.message}`)
        } else {
          source.cancel(error.message) // mark cancellation for all token watchers
        }
      })
  }

  function getChannels() {
    return get('https://reqres.in/api/users?page=1&delay=30'); // delayed 30 secs
  }
  function getContacts() {
    return get('https://reqres.in/api/users?page=2'); // no delay
  }
  function getEventActions() {
    return get('https://httpbin.org/status/401'); // 401 - auth error
  }

  ...
}

В вашем перехватчике вы также проигнорируете ошибки отмены запроса:

export default (http, store, router) => {
  http.interceptors.response.use(
    response => response,
    error => {
      if (http.isCancel(error)) {
        return Promise.reject(error)
      }

      ...

      // show notification here
    }
}

demo

0 голосов
/ 01 апреля 2019

В качестве альтернативы отмене Axios вы можете использовать Обещание Bluebird Promise , что проще.

Преимущества новой отмены по сравнению со старой отменой:

  • .cancel () является синхронным.
  • код настройки не требуется для работы отмены
  • сочетаются с другими функциями bluebird, такими как Promise.all

ЗдесьдемоЯ добавил некоторые записи в axios.get(...).then(...) для отслеживания завершения каждого вызова.

Закомментируйте строку promises.forEach(p => p.cancel()), чтобы убедиться, что без отмены безошибочные вызовы будут выполнены до завершения.

//for demo, check if fetch completes 
const logCompleted = (res) => console.log(`Promise completed, '${res.config.url}'`) 

function getChannels() {
  return axios.get("https://reqres.in/api/users?page=1&delay=5").then(logCompleted)
}
function getContacts() {
  return axios.get("https://reqres.in/api/users?page=2").then(logCompleted)
}
function getEventActions() {
  return axios.get("https://httpbin.org/status/401").then(logCompleted)
}

Promise.config({ cancellation: true }); // Bluebird config
window.Promise = Promise; // axios promises are now Bluebird flavor

const promises = [getChannels(), getContacts(), getEventActions()];
Promise.all(promises)
  .then(([channels, contacts, eventActions]) => {
    console.log('Promise.all.then', { channels, contacts, eventActions });
  })
  .catch(err => {
    console.log(`Promise.all.catch, '${err.message}'`)
    promises.forEach(p => p.cancel());
  })
  .finally(() => console.log('Promise.all.finally'))
<script src="https://cdnjs.cloudflare.com/ajax/libs/axios/0.18.0/axios.min.js"></script>
<script src="https://cdn.jsdelivr.net/bluebird/latest/bluebird.core.min.js"></script>

Почему это работает

Promise.all () вместо axios.all ()

Глядя на эту старую проблему с axios Удалить axios.all и axios.spread # 1042 можно увидеть

Axios использует Promise.all под капотом ...

и этот

axios.all([getUserAccount(), getUserPermissions()])
  .then(axios.spread(function (acct, perms) {
    // Both requests are now complete
}));

может быть заменен этим

Promise.all([getUserAccount(), getUserPermissions()])
  .then(function ([acct, perms]) {
    // Both requests are now complete
});

, поэтому мы можем перейти к работе с Promises напрямую и при этом иметь те же функции.


Обещания не выполняются быстро

С MDN мы видим

Promise.all отклоняется, если любой изэлементы отклонены.Например, если вы передаете четыре обещания, которые разрешаются после истечения времени ожидания, и одно обещание, которое сразу же отклоняется, то Promise.all будет отклонено немедленно.

, поэтому в этом шаблоне

Promise.all(...)
.then(...)
.catch(...);

.catch() сработает, когда первое обещание не будет выполнено (в отличие от then(), который ожидает завершения всех обещаний).


Составление Promise.all и .cancel()

Шаблон довольно прост, просто отмените все обещания в .catch() (который вызывается при первой ошибке).

Ссылка на этот вопрос для деталей Остановите другие обещания, когда Promise.all () отклоняет


Замена Bluebird в хранилище Vue

Это базовая реализация в Vuex.

yarn add bluebird
import Vue from "vue";
import Vuex from "vuex";
import axios from "axios";
import Promise from 'bluebird';
Vue.use(Vuex);

Promise.config({ cancellation: true }); // Bluebird config
window.Promise = Promise; // axios promises are now Bluebird flavor

export default new Vuex.Store({
  actions: {
    fetchData({ dispatch }) {
      function getChannels() {
        return axios.get("https://reqres.in/api/users?page=1&delay=5");
      }
      function getContacts() {
        return axios.get("https://reqres.in/api/users?page=2");
      }
      function getEventActions() {  // 401 - auth error
        return axios.get("https://httpbin.org/status/401");
      }

      const promises = [getChannels(), getContacts(), getEventActions()];
      Promise.all(promises)
        .then(([channels, contacts, eventActions]) => {
          dispatch("channels/setChannels", channels.data, { root: true });
          dispatch("contacts/setContacts", contacts.data, { root: true });
          dispatch("events/setActions", eventActions.data, { root: true });
        })
        .catch(err => {
          promises.forEach(p => p.cancel());
        })
    }
  }
});
0 голосов
/ 01 марта 2019

РЕДАКТИРОВАТЬ: @ ответ tony19 гораздо лучше, поскольку позволяет отменять запросы, ожидающие обработки после первой ошибки, и не требует дополнительной библиотеки.


Одним из решений будет назначение уникального идентификатора (в этом примере я буду использовать пакет uuid/v4, не стесняйтесь использовать что-то еще) для всех запросов, которые вы используете одновременно:

import uuid from 'uuid/v4'

const actions = {
    fetchData({commit, dispatch}) {
        const config = {
            _uuid: uuid()
        }

        function getChannels() {
            return http.get('channels', config)
        }

        function getContacts() {
            return http.get('conversations', config)
        }

        function getEventActions() {
            return http.get('events/actions', config)
        }

        // 20 more functions calls

        axios.all([
            getChannels(),
            getContacts(),
            getEventActions()
        ]).then(axios.spread(function (channels, contacts, eventActions) {
            dispatch('channels/setChannels', channels.data, {root: true})
            dispatch('contacts/setContacts', contacts.data, {root: true})
            dispatch('events/setActions', eventActions.data, {root: true})
        }))
    }
}

Затем в перехватчике вы можете выбрать обработку ошибки один раз, используя этот уникальный идентификатор:

export default (http, store, router) => {
    // Here, you create a variable that memorize all the uuid that have
    // already been handled
    const handledErrors = {}
    http.interceptors.response.use(response => response, (error) => {
        // Here, you check if you have already handled the error
        if (error.config._uuid && handledErrors[error.config._uuid]) {
            return Promise.reject(error)
        }

        // If the request contains a uuid, you tell 
        // the handledErrors variable that you handled
        // this particular uuid
        if (error.config._uuid) {
            handledErrors[error.config._uuid] = true
        }

        // And then you continue on your normal behavior

        const {response} = error;

        let message = 'Ops. Algo de errado aconteceu...';

        if([401].indexOf(response.status) > -1){
            localforage.removeItem('token');

            router.push({
                name: 'login'
            });

            Vue.notify({
                group: 'panel',
                type: 'error',
                duration: 5000,
                text: response.data.message ? response.data.message : message
            });
        }

        return Promise.reject(error);
    })
}

Дополнительное примечание: вы можете упростить эту функцию fetchData до этого:

const actions = {
    fetchData({commit, dispatch}) {
        const config = {
            _uuid: uuid()
        }

        const calls = [
            'channels',
            'conversations',
            'events/actions'
        ].map(call => http.get(call, config))

        // 20 more functions calls

        axios.all(calls).then(axios.spread(function (channels, contacts, eventActions) {
            dispatch('channels/setChannels', channels.data, {root: true})
            dispatch('contacts/setContacts', contacts.data, {root: true})
            dispatch('events/setActions', eventActions.data, {root: true})
        }))
    }
}
...