AzureEnvironment type is missing properties for endpoints

[chrmarti]

This looks like an omission, aren't these properties part of the API?

[amarzavery]

Can you provide information about the Environment and the properties that are missing?

[chrmarti]

The class declaration in

azure-sdk-for-node/runtime/ms-rest-azure/index.d.ts

Line 167 in fb1d276

export class AzureEnvironment {

shows the properties being passed as the parameters object. I assume the class should have the same properties so these can be read. Examples are: managementEndpointUrl, activeDirectoryEndpointUrl, activeDirectoryResourceId.

[amarzavery]

Ah I see. Got your point. Will fix that in the upcoming release.
On a side note, in the actual javascript code we are adding some supported environments to the prototype of AzureEnvironment class. https://github.com/Azure/azure-sdk-for-node/blob/master/runtime/ms-rest-azure/lib/azureEnvironment.js#L156 . I was wondering how would I add those supported environments to the class definition in d.ts file.

[chrmarti]

I think you can add static properties to do that.

[amarzavery]

This has been fixed with 2.3.0 version of ms-rest-azure.