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

Add "x-ms-is-hns-enabled" to ServiceGetAccountInfoResponse #12530

Closed
JasonYeMSFT opened this issue Nov 13, 2020 · 4 comments · Fixed by #12722
Closed

Add "x-ms-is-hns-enabled" to ServiceGetAccountInfoResponse #12530

JasonYeMSFT opened this issue Nov 13, 2020 · 4 comments · Fixed by #12722
Assignees
@ljian3377
Labels
Client This issue points to a problem in the data-plane of the library. cross-language alignment mark issues need to be aligned across different languages feature-request This issue requires a new behavior in the product in order be resolved. Storage Storage Service (Queues, Blobs, Files)
Milestone
[2020] December

Comments

@JasonYeMSFT
Copy link
Contributor

Is your feature request related to a problem? Please describe.
The "x-ms-is-hns-enabled" is not exposed via ServiceGetAccountInfoResponse.

Describe the solution you'd like
Expose the response value there.

Describe alternatives you've considered
N/A

Additional context
Currently the only way to access it is to call response._response.headers.get("x-ms-is-hns-enabled")
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Nov 13, 2020
@JasonYeMSFT JasonYeMSFT mentioned this issue Nov 13, 2020
Closed
@ramya-rao-a
Copy link
Contributor

Hey @JasonYeMSFT

Am guessing this issue is for one of our storage packages, can you share which one?

@ramya-rao-a ramya-rao-a added Client This issue points to a problem in the data-plane of the library. Storage Storage Service (Queues, Blobs, Files) labels Nov 13, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Nov 13, 2020
@JasonYeMSFT
Copy link
Contributor Author

Yeah, it is @azure/storage-blob

@ljian3377 ljian3377 added the feature-request This issue requires a new behavior in the product in order be resolved. label Nov 16, 2020
@ljian3377 ljian3377 added this to the [2020] December milestone Nov 16, 2020
@ljian3377 ljian3377 added the cross-language alignment mark issues need to be aligned across different languages label Nov 30, 2020
@ljian3377 ljian3377 mentioned this issue Dec 1, 2020
Merged
@lmazuel
Copy link
Member

lmazuel commented Dec 1, 2020

@ramya-rao-a Swagger fixed Azure/azure-rest-api-specs#11871

@ljian3377
Copy link
Member

Added isHierarchicalNamespaceEnabled to the response of BlobServiceClient.getAccountInfo().

This is now availabe in @azure/storage-blob@12.4.0-beta.1 and the it will be GA next month.

openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this issue Jan 26, 2021
CodeGen from PR 12530 in Azure/azure-rest-api-specs
53d893f 
On branch edburns-msft-broken-link-ssh-keys (Azure#12530)

modified:   specification/compute/resource-manager/Microsoft.Compute/stable/2020-06-01/compute.json

Fix broken link at `definitions.SshPublicKey.keyData.description`.
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ljian3377 ljian3377

Labels
Client This issue points to a problem in the data-plane of the library. cross-language alignment mark issues need to be aligned across different languages feature-request This issue requires a new behavior in the product in order be resolved. Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Milestone
[2020] December
Development

Successfully merging a pull request may close this issue.

[storage] Add "x-ms-is-hns-enabled" to ServiceGetAccountInfoResponse ljian3377/azure-sdk-for-js
4 participants
@lmazuel @ljian3377 @ramya-rao-a @JasonYeMSFT