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

Fail to create one ADLS Gen2 folder with name contains '%' #3987

Closed
v-xianya opened this issue Jan 19, 2021 · 2 comments
Closed

Fail to create one ADLS Gen2 folder with name contains '%' #3987

v-xianya opened this issue Jan 19, 2021 · 2 comments
Assignees
@JasonYeMSFT
Labels
🪲 regression Issue was working in a previous version ⚙️ adls gen2 Related to hierarchical namespaces (ADLS Gen 2) 🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Milestone
1.18.0

Comments

@v-xianya
Copy link
Member

v-xianya commented Jan 19, 2021 •
edited
Loading

Storage Explorer Version: 1.17.0
Build Number: 20210119.2
Branch: main
Platform/OS: Windows 10/ CentOS 7.6.1810 (Core)/ MacOS Catalina
Architecture: ia32/x64
Regression From: Previous release (1.17.0)

Steps to Reproduce

  1. Expand one ADLS Gen2 storage account -> Blob Containers.
  2. Create a blob container -> Create a folder with name contains '%'.
  3. Check whether the folder can be successfully created.

Expected Experience

Succeed to create the folder.
image

Actual Experience

Fail to create the folder.
image
image

Additional Context

Succeed to create a folder with name contains '%' from Portal.
@v-xianya v-xianya added 🧪 testing Found through regular testing 🪲 regression Issue was working in a previous version ⚙️ adls gen2 Related to hierarchical namespaces (ADLS Gen 2) labels Jan 19, 2021
@JasonYeMSFT JasonYeMSFT self-assigned this Jan 19, 2021
@JasonYeMSFT JasonYeMSFT added this to the 1.18.0 milestone Jan 19, 2021
@JasonYeMSFT
Copy link
Contributor

JasonYeMSFT commented Jan 19, 2021 •
edited
Loading

The error is an SDK internal error. The cause is that decodeURIComponent("New%Folder") throws. Maybe we should encode the folder names before passing to the SDK.

@JasonYeMSFT JasonYeMSFT added the ❔ external Root cause of this issue is in another component, product, or service label Jan 19, 2021
@JasonYeMSFT JasonYeMSFT removed the ❔ external Root cause of this issue is in another component, product, or service label Jan 19, 2021
@JasonYeMSFT JasonYeMSFT added the ✅ merged A fix for this issue has been merged label Jan 26, 2021
@v-xianya
Copy link
Member Author

Verified on build 20210127.3. Fixed.

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

@JasonYeMSFT JasonYeMSFT

Labels
🪲 regression Issue was working in a previous version ⚙️ adls gen2 Related to hierarchical namespaces (ADLS Gen 2) 🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Projects
None yet
Milestone
1.18.0
Development

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