storj / edge

Storj edge services (including multi-tenant, S3-compatible server to interact with the Storj network)
GNU Affero General Public License v3.0
53 stars 18 forks source link

Address unresolved comments from Object Lock changes #461

Closed NiaStorj closed 1 month ago

NiaStorj commented 3 months ago

Summary:

Address feedback on code reviews for Object Lock

Document:

https://docs.google.com/document/d/1Xk5dd7FVAkz3jyUz6XZ4Zn56jEmtEYPmJZ8HN8MMJ2U/edit

### Tasks
- [x] https://review.dev.storj.io/c/storj/storj/+/13759
- [x] https://review.dev.storj.io/c/storj/storj/+/13871
- [x] https://review.dev.storj.io/c/storj/storj/+/13872
- [x] https://review.dev.storj.io/c/storj/storj/+/13876
- [x] https://review.dev.storj.io/c/storj/storj/+/13890
- [x] https://review.dev.storj.io/c/storj/storj/+/13878
- [x] https://review.dev.storj.io/c/storj/storj/+/13880
- [x] https://review.dev.storj.io/c/storj/storj/+/13877
storj-gerrit[bot] commented 3 months ago

Change satellite/metabase: fix SetObjectRetention race condition mentions this issue.

storj-gerrit[bot] commented 3 months ago

Change satellite/metainfo: refactor simple validation of request fields mentions this issue.

storj-gerrit[bot] commented 3 months ago

Change satellite/metabase: fix data race when deleting locked objects mentions this issue.