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

Undelete Container API creates a new container #2318

Open
anna-mazhar opened this issue Dec 8, 2023 · 1 comment
Open

Undelete Container API creates a new container #2318

anna-mazhar opened this issue Dec 8, 2023 · 1 comment

Comments

@anna-mazhar
Copy link

anna-mazhar commented Dec 8, 2023

Which service(blob, file, queue, table) does this issue concern?

Azure Blob

Which version of the Azurite was used?

3.21.0

Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)

npm

What's the Node.js version?

14.18.1

What problem was encountered?

Undelete container API does not behave correctly. A few scenarios of API violations:

  • When a container previously deleted is undeleted, the API generates an empty container with the same name rather than restoring its prior contents.
  • If attempting to undelete a container that was never deleted, the API generates an empty container with that name instead of indicating that it was never deleted.
  • The undelete container cloud API enforces strict verification for the "delete container version," while Azurite accepts any value for it.

From the logs, it looks like the create_container operation is performed against this API.

Steps to reproduce the issue?

Logs to reproduce the scenario 1: debug.log

@EmmaZhu
Copy link
Member

EmmaZhu commented Dec 12, 2023

Hi @anna-mazhar ,

Undelete container is not supported yet.

Older version Azurite cannot recognize container_restore request, it treat the request as a create request from the method and URI. Latest Azurite can recognize container_restore request, and can report "Current API is not implemented yet." error.

Can you please upgrade to the latest Azurite?

Thanks
Emma

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