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

ObjectStore links/updates #155

Open
5 of 11 tasks
aricart opened this issue Sep 19, 2022 · 3 comments
Open
5 of 11 tasks

ObjectStore links/updates #155

aricart opened this issue Sep 19, 2022 · 3 comments
Assignees
Labels
client Client related work enhancement New feature or request

Comments

@aricart
Copy link
Member

aricart commented Sep 19, 2022

Overview

This issue documents some updates made to the Go client object store implementation to handle some conditions reasonably.

  • update the meta of a deleted object should not be allowed by the client.
  • updating the meta to point to an already existing name should not be allowed
  • put should not allow setting the meta.link information - this is only supported via the link API
  • getInfo should not return info for deleted objects. Optional methods can be added to support getting info on deleted objects.
  • list should not return info for deleted objects. Optional methods can be added to support getting info on deleted objects.

ObjectStore ADR nats-io/nats.deno@6164d01

Clients and Tools

Other Tasks

Client authors please update with your progress. If you open issues in your own repositories as a result of this request, please link them to this one by pasting the issue URL in a comment or main issue description.

@scottf
Copy link
Collaborator

scottf commented Sep 19, 2022

@scottf
Copy link
Collaborator

scottf commented Sep 19, 2022

updating the meta to point to an already existing name should not be allowed

It's an error to update an object's name to an existing and not deleted object. You can update an objects name to an existing but deleted object.

@scottf
Copy link
Collaborator

scottf commented Sep 19, 2022

put should not allow setting the meta.link information - this is only supported via the link API

The put method should check the incoming meta for a link and fail with error if it finds one.

@piotrpio piotrpio removed their assignment Sep 22, 2022
@scottf scottf removed their assignment Sep 30, 2022
@scottf scottf assigned levb and unassigned ColinSullivan1 Mar 21, 2023
@ripienaar ripienaar removed their assignment Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client Client related work enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

10 participants