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

Error in description of "az storage account file-service-properties update" SMB properties #28069

Closed
hcrutz opened this issue Dec 20, 2023 · 1 comment · Fixed by #28115
Closed

Error in description of "az storage account file-service-properties update" SMB properties #28069

hcrutz opened this issue Dec 20, 2023 · 1 comment · Fixed by #28115
Assignees
@evelyn-ys @calvinhzy
Labels
Auto-Assign Auto assign by bot Azure CLI Team The command of the issue is owned by Azure CLI team customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that Storage az storage
Milestone
Backlog

Comments

@hcrutz
Copy link

hcrutz commented Dec 20, 2023

Describe the bug

The documentation contains the following section:
"Set secured SMB setting for file service.
az storage account file-service-properties update --versions SMB2.1;SMB3.0;SMB3.1.1 --auth-
methods NTLMv2;Kerberos --kerb-ticket-encryption RC4-HMAC;AES-256 --channel-encryption AES-
CCM-128;AES-GCM-128;AES-GCM-256 -n mystorageaccount -g MyResourceGroup"
But the listed values for --channel-encryption are wrong and cause an error if you use them as listed

Related command

az storage account file-service-properties update --versions SMB3.1.1 --auth-methods Kerberos --kerb-ticket-encryption AES-256
--channel-encryption AES-GCM-256 -n mystorageaccount -g MyResourceGroup

Errors

(InvalidXmlDocument) XML specified is not syntactically valid.
RequestId:40331b1e-901a-0000-7d54-33bb39000000
Time:2023-12-20T14:57:16.6027100Z
Code: InvalidXmlDocument
Message: XML specified is not syntactically valid.
RequestId:40331b1e-901a-0000-7d54-33bb39000000
Time:2023-12-20T14:57:16.6027100Z

Issue script & Debug output

not needed here

Expected behavior

The values you have to enter here for --channel-encryption are:
AES-128-CCM;AES-128-GCM;AES-256-GCM

Environment Summary

azure-cli 2.55.0

core 2.55.0
telemetry 1.1.0

Dependencies:
msal 1.24.0b2
azure-mgmt-resource 23.1.0b2

Additional context

No response
@hcrutz hcrutz added the bug This issue requires a change to an existing behavior in the product in order to be resolved. label Dec 20, 2023
@yonzhan
Copy link
Collaborator

yonzhan commented Dec 20, 2023

Thank you for opening this issue, we will look into it.

@microsoft-github-policy-service microsoft-github-policy-service bot added customer-reported Issues that are reported by GitHub users external to the Azure organization. Storage az storage labels Dec 20, 2023
@microsoft-github-policy-service microsoft-github-policy-service bot added Auto-Assign Auto assign by bot Azure CLI Team The command of the issue is owned by Azure CLI team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Dec 20, 2023
@yonzhan yonzhan added this to the Backlog milestone Dec 20, 2023
@yonzhan yonzhan removed the bug This issue requires a change to an existing behavior in the product in order to be resolved. label Dec 20, 2023
@calvinhzy calvinhzy mentioned this issue Jan 2, 2024
Merged
3 tasks
calvinhzy added a commit that referenced this issue Jan 16, 2024
@calvinhzy
{Storage} Fix #28069: `az storage account file-service-properties up…
6040420 
…date`: Fix `--channel-encryption` in example (#28115)
MaxHorstmann pushed a commit to MaxHorstmann/azure-cli that referenced this issue Jan 19, 2024
@calvinhzy @MaxHorstmann
{Storage} Fix Azure#28069: `az storage account file-service-properti…
f328b5c 
…es update`: Fix `--channel-encryption` in example (Azure#28115)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@evelyn-ys evelyn-ys

@calvinhzy calvinhzy

Labels
Auto-Assign Auto assign by bot Azure CLI Team The command of the issue is owned by Azure CLI team customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that Storage az storage
Projects
None yet
Milestone
Backlog
4 participants
@evelyn-ys @hcrutz @calvinhzy @yonzhan