upload-api should upgrade to `@ucanto/transport` that has more verbose error messages from HTTP transport non-ok responses - Githubissues

storacha-network / w3infra

🏗️ Infra for the w3up UCAN protocol implementation

Other

15 stars 5 forks source link

upload-api should upgrade to `@ucanto/transport` that has more verbose error messages from HTTP transport non-ok responses #336

Closed gobengo closed 2 months ago

gobengo commented 6 months ago

Goal:

if/when this is released, w3infra should depend on it https://github.com/web3-storage/ucanto/pull/343
- to get https://github.com/web3-storage/ucanto/commit/d5f6916a41447bcc32bfbd9378da2b4779ca2136

Why?

this will make errors in w3up that come from w3infra (really any database-interaction failures) be easier to differentiate by error message, e.g. in tools like sentry. It should help us find errors faster, understand the cause, and prevent them from happening again.
- e.g. having the connection url href in here will group all these http errors by the URL of the service that responded with non-ok

### Tasks
- [ ] https://github.com/web3-storage/ucanto/pull/343
- [x] PR bumping dep in w3up https://github.com/web3-storage/w3up/pull/1312
- [ ] PR bumping deps in w3infra
- [ ] release w3infra to prod after PR merge

vasco-santos commented 2 months ago

api uses latest @ucanto/transport@9.1.1 now