Error raised if someone goes offline and attempts a request

Hi! I’m running into an issue where, if someone attempts a request while they are offline, result.headers on line 40 blows up because err.response is undefined. I’m unsure of the proper way to go about fixing this, or if there is something I should be doing with the axios instance before passing it to axios-fetch to avoid this issue altogether.

Any help would be greatly appreciated!

[mdlavin]

The simplest thing I can think of that might work is to change:

} catch (err) {
  result = err.response;
}

into

} catch (err) {
  // Non-HTTP failures should be re-thrown. This will handle lower level network failures
  if (!err.response) throw err;
  // HTTP level failures can be translated into fetch style responses
  result = err.response;
}

Is it possible for you to try a change like that and see if it fixes your problem?

[jkdowdle]

Unless someone feels otherwise going to go ahead and close this issue. The proposed solution was implemented in https://github.com/lifeomic/axios-fetch/pull/78 and released in @lifeomic/axios-fetch@2.0.2 https://github.com/lifeomic/axios-fetch/pull/79