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

File delete fails for files in the format <GUID>##9999.csv #7916

Closed
3 tasks done
maps-mike opened this issue May 1, 2024 · 11 comments
Closed
3 tasks done

File delete fails for files in the format <GUID>##9999.csv #7916

maps-mike opened this issue May 1, 2024 · 11 comments
Assignees
Labels
⚙️ files Related to file storage ✅ merged A fix for this issue has been merged
Milestone

Comments

@maps-mike
Copy link

maps-mike commented May 1, 2024

Preflight Checklist

Storage Explorer Version

1.33.1

Regression From

No response

Architecture

x64

Storage Explorer Build Number

20240410.2

Platform

Windows

OS Version

Windows 10

Bug Description

File delete fails for files in the format ##9999.csv

Steps to Reproduce

Create ADLS G2 storage account
create folder
add a file called ab9f89ba-f069-44b1-84a6-7dda8da39147##7587.csv
try to delete file using storage explorer

Actual Experience

Error message in storage explorer:

Deletion of 'ab9f89ba-f069-44b1-84a6-7dda8da39147##7587.csv' from 'landing/Transactional/upload_api_sandbox/20240430/' failed: 0 completed, 1 failed (used name and key)
Started at: 01/05/2024 08:17, Duration: 4 seconds

Error in log:

2024/05/01 07:17:14 ERR: [P#0-T#0] https://adlsdatahubraw.dfs.core.windows.net/landing/Transactional/upload_api_sandbox/20240430/ab9f89ba-f069-44b1-84a6-7dda8da39147%23%237587.csv: 404: DELETE ERROR -404 The specified path does not exist.. X-Ms-Request-Id:4666f2af-401f-000c-0997-9b4c15000000

Expected Experience

File is deleted

Additional Context

Other files can be deleted without issue it is just files in the format ([GUID]##9999.csv) above that cannot be deleted.

The problem files can still be deleted by other mechanisms - I usually use the Azure portal.

@craxal craxal added this to the 1.34.0 milestone May 1, 2024
@craxal
Copy link
Contributor

craxal commented May 3, 2024

@maps-mike My guess is the double hash (##) is what's causing the issue. Some characters need special handling. Strange, though, as the URL in the error message looks correct. Can you confirm that URL of the blob used by Storage Explorer is the same one used by the Azure portal? You can check this by looking at the blob's properties both in Storage Explorer and the portal.

@craxal
Copy link
Contributor

craxal commented May 3, 2024

@maps-mike I am not able to reproduce this issue, either by deleting one or multiple blobs. Can you add any more details about what exactly you were doing? When you refresh the explorer view, are the blobs you're trying to delete still present?

@craxal craxal assigned craxal and unassigned richardMSFT May 3, 2024
@craxal craxal added ❔ more info More information is needed from issue author to proceed ⚙️ files Related to file storage labels May 6, 2024
@craxal
Copy link
Contributor

craxal commented May 10, 2024

@maps-mike Have you had a chance to reproduce this issue and gather more information? Anything you can provide can help us investigate further.

@maps-mike
Copy link
Author

maps-mike commented May 13, 2024 via email

@craxal
Copy link
Contributor

craxal commented May 13, 2024

@maps-mike For testing purposes, can you upload and delete some files with some variations? For example, does the issue still reproduce if:

  • You only have one "#" character
  • You have no "#" characters
  • You change the directory structure, such as moving the file to the root
  • You delete multiple files at once

Strictly speaking, "#" characters are not legal in a URL path. Azure portal might be un-escaping characters for display convenience.

The fact that I still can't repo the issue suggests we are escaping the "#" characters correctly:

./azcopy remove "https://XXX.dfs.core.windows.net/test01/01234567-89ab-cdef-0123-456789abcdef%23%231234.csv" --from-to=BlobFSTrash --recursive --log-level=INFO;

@maps-mike
Copy link
Author

maps-mike commented May 14, 2024 via email

@craxal
Copy link
Contributor

craxal commented May 14, 2024

@maps-mike Thank you for that information!

Ah ha! Sorry, I should have tried this first: I was able to reproduce the issue using our 1.33.0 release. It doesn't reproduce in our working builds now, so I suspect the issue has already been addressed (by updating our version of AzCopy, probably).

Here is a link to a private build. Can you download, install, and give it a try to see if the problem persists?

[LINK REMOVED]

@maps-mike
Copy link
Author

maps-mike commented May 15, 2024 via email

@craxal
Copy link
Contributor

craxal commented May 21, 2024

@maps-mike Any progress? If you need a new link, please let me know.

@maps-mike
Copy link
Author

maps-mike commented May 22, 2024 via email

@craxal
Copy link
Contributor

craxal commented May 22, 2024

Excellent! Our next release (1.34.0) contains an AzCopy update, so that means the next release should resolve your issue.

@craxal craxal added ✅ merged A fix for this issue has been merged and removed ❔ more info More information is needed from issue author to proceed labels May 22, 2024
@craxal craxal closed this as completed May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ files Related to file storage ✅ merged A fix for this issue has been merged
Projects
None yet
Development

No branches or pull requests

3 participants