Asynchronous support: onRequest/onResponse

Hi, is it possible to support “async for extensions before and after communication”?

Corresponding part https://github.com/nuxt-community/axios-module/blob/22a27a4cdf8c60b065a5cbae68e749da69bf9974/lib/plugin.js#L23-L28

Use Case We are using JWT of Firebase Authentication. We would like to extend the process of granting “Bearer tokens” for all or specific communications prior to communications. If the JWT has been expired, the method of getIdToken() updates it to make it a valid token before continuing with the request. Here, communication occurs before communication, and asynchronous processing should be required.

How to fix We have confirmed the operation with the following support, at least in my environment. We cloned this project and passed the test.

lib/plugin.js

  onRequest(fn) {
    this.interceptors.request.use(async config => await fn(config) || config)
  },
  onResponse(fn) {
    this.interceptors.response.use(async response => await fn(response) || response)
  },

Please update as a bug(?) fix if possible, Thank you in advance.

[areindl]

I came across the same error message when using an async function in onRequest. I think the problem might be that async functions by default return a promise (see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/async_function#return_value) but here you need to return the config object to pass it further to Axios.

Try something like this:

$axios.onRequest(async function (config) {
    try {
            await [SOMETHING]
            console.log('Awaited function finished')
        } catch (error) {
            console.log(error)
        }
    return config
  })

It took me a while to realise because the docs say “these functions don’t have to return anything by default” (https://axios.nuxtjs.org/helpers) but I guess the implicitly returned promise overrides the config object which would otherwise be passed on in the background by default.

You Sir, are a legend! That should definitely be in the docs.

[insell824]

Thank you for your reply.

The code as instructed worked fine.

Just in case, I shared the sample code that I forked and modified. https://codesandbox.io/s/priceless-cartwright-5sgj7?file=/plugins/axios-extension.js

Thank you so much!