New upload code path is dropping satellite errors

storj / uplink

Storj network Go library

MIT License

115 stars 18 forks source link

New upload code path is dropping satellite errors #154

Closed mniewrzal closed 9 months ago

mniewrzal commented 10 months ago

While uploading objects some limits error can happen e.g. segment limit error. At the moment in such case libuplink is returning things like context canceled error instead of uplink.ErrSegmentsLimitExceeded

Acceptance Criteria:

return correct errors while uploading object

storj-gerrit[bot] commented 10 months ago

Change private/storage/streams: fix upload errors handling mentions this issue.

mniewrzal commented 9 months ago

https://review.dev.storj.io/c/storj/uplink/+/11170

mniewrzal commented 9 months ago

Issue should be fixed with above fix.

iglesiasbrandon commented 9 months ago

i think we can close this issues since its been deployed @mniewrzal @shaupt131