Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
gobengo opened this issue Feb 21, 2024 · 1 comment
Assignees

Comments

@gobengo
Copy link
Contributor

gobengo commented Feb 21, 2024

Goal:

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
@gobengo gobengo self-assigned this Feb 21, 2024
@vasco-santos
Copy link
Contributor

api uses latest @ucanto/transport@9.1.1 now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants