Replace axios with a simple custom fetch wrapper

16 points
1/21/1970
2 days ago
by mariuz

Comments


hackerman70000

Every team I've seen write a fetch wrapper eventually reimplements half of axios, just worse. The article works for a bookshelf demo app. In production you'll want retries, timeouts, request deduplication, cancellation - and suddenly your "simple wrapper" is a library with no tests and no docs

18 hours ago

pheew

I’m all for replacing as much 3rd party libs with stdlib but it’s hard to look past the storing of the jwt in localStorage. Please don’t do that people. It’s very easily extracted through xss attacks.

a day ago

moi2388

A cookie is susceptible to both xss and CSRF. You ought to be protecting against xss anyway.

a day ago

joeframbach

In the case of authentication redirect, any inflight request is going to fail and you're about to navigate in a few milliseconds, so I don't resolve those promises. Return new Promise(() => {}).

2 days ago