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.

Sign up for GitHub

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

Jump to bottom

There is a successful activity log when deleting one table under a SAS attached Storage account (with Read, List permissions) #5618

Closed
v-xianya opened this issue Apr 13, 2022 · 2 comments
Closed

There is a successful activity log when deleting one table under a SAS attached Storage account (with Read, List permissions) #5618

v-xianya opened this issue Apr 13, 2022 · 2 comments
Assignees
@craxal
Labels
🪲 regression Issue was working in a previous version ⚙️ attach Related to attaching via keys, SAS, etc. ⚙️ tables Related to table storage ❔ external Root cause of this issue is in another component, product, or service 🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Milestone
1.24.0

Comments

@v-xianya
Copy link
Member

Storage Explorer Version: 1.24.0-dev
Build Number: 20220413.3
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.3 (Apple M1 Pro)
Architecture ia32\x64
How Found: From running test cases
Regression From: Previous release (1.23.0)

Steps to Reproduce

  1. Expand one storage account -> Tables.
  2. Create a table.
  3. Attach the storage account via SAS URL with permissions 'Read, List'.
  4. Expand the attached storage account -> Tables.
  5. Select one table -> Delete it.
  6. Check there is a failed activity log.

Expected Experience

There is a failed activity log.
image

Error:
"name": "RestError", "message": "{\"odata.error\":{\"code\":\"AuthorizationPermissionMismatch\",\"message\":{\"lang\":\"en-US\",\"value\":\"This request is not authorized to perform this operation using this permission.\\nRequestId:bcb1db7d-7002-0024-2f0e-4f67f8000000\\nTime:2022-04-13T08:13:49.7130361Z\"}}}",

Actual Experience

  1. There is a successful activity log.
    image

  2. The table disappears from the tree view.

Additional Context

The table displays on the tree view after manually refreshing.
@v-xianya v-xianya added 🧪 testing Found through regular testing ⚙️ tables Related to table storage ⚙️ attach Related to attaching via keys, SAS, etc. 🪲 regression Issue was working in a previous version labels Apr 13, 2022
@craxal craxal self-assigned this Apr 13, 2022
@craxal craxal added this to the 1.24.0 milestone Apr 13, 2022
@craxal
Copy link
Contributor

craxal commented Apr 13, 2022

That's weird. The SDK isn't throwing an error even though the logs clearly show that the operation failed. Looks like the regression is in the SDK, as the expected error is thrown in slightly older versions of the SDK.

See Azure/azure-sdk-for-js#21385.

@craxal craxal added ❔ external Root cause of this issue is in another component, product, or service ✅ merged A fix for this issue has been merged labels Apr 13, 2022
@craxal craxal closed this as completed Apr 15, 2022
@v-xianya
Copy link
Member Author

Verified this issue on the build 20220418.1. Fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@craxal craxal

Labels
🪲 regression Issue was working in a previous version ⚙️ attach Related to attaching via keys, SAS, etc. ⚙️ tables Related to table storage ❔ external Root cause of this issue is in another component, product, or service 🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Projects
None yet
Milestone
1.24.0
Development

No branches or pull requests
2 participants
@craxal @v-xianya