update tracing for consistency

sdk/identity/identity/CHANGELOG.md

@@ -100,6 +100,8 @@ Azure Service Fabric support hasn't been added on the initial version 2 of Ident
   - Identity v2 also removes the `postLogoutRedirectUri` from the options to the constructor for `InteractiveBrowserCredential`. This option wasn't being used. Instead of using this option, use MSAL directly. For more information, see [Authenticating with the @azure/msal-browser Public Client](https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/identity/identity/samples/AzureIdentityExamples.md#authenticating-with-the-azuremsal-browser-public-client).
   - In Identity v2, `VisualStudioCodeCredential` throws a `CredentialUnavailableError` unless the new [@azure/identity-vscode](https://www.npmjs.com/package/@azure/identity-vscode) plugin is used.
 
+- Standardizing the tracing span names to be `<className>.<operationName>` over `<className>-<operationName>`
+
 #### Breaking Changes from 2.0.0-beta.4
 
 - Removed the `allowMultiTenantAuthentication` option from all of the credentials. Multi-tenant authentication is now enabled by default. On Node.js, it can be disabled with the `AZURE_IDENTITY_DISABLE_MULTITENANTAUTH` environment variable.

sdk/identity/identity/src/credentials/azureCliCredential.ts

@@ -117,7 +117,7 @@ export class AzureCliCredential implements TokenCredential {
 
     let responseData = "";
 
-    const { span } = createSpan("AzureCliCredential-getToken", options);
+    const { span } = createSpan("AzureCliCredential.getToken", options);
 
     try {
       const obj = await cliCredentialInternals.getAzureCliAccessToken(resource, tenantId);

sdk/identity/identity/src/credentials/chainedTokenCredential.ts

@@ -60,7 +60,7 @@ export class ChainedTokenCredential implements TokenCredential {
     let successfulCredentialName = "";
     const errors = [];
 
-    const { span, updatedOptions } = createSpan("ChainedTokenCredential-getToken", options);
+    const { span, updatedOptions } = createSpan("ChainedTokenCredential.getToken", options);
 
     for (let i = 0; i < this._sources.length && token === null; i++) {
       try {

sdk/identity/identity/src/credentials/clientSecretCredential.browser.ts

@@ -66,7 +66,7 @@ export class ClientSecretCredential implements TokenCredential {
     options?: GetTokenOptions
   ): Promise<AccessToken | null> {
     const { span, updatedOptions: newOptions } = createSpan(
-      "ClientSecretCredential-getToken",
+      "ClientSecretCredential.getToken",
       options
     );
 

sdk/identity/identity/src/credentials/managedIdentityCredential/index.ts

@@ -136,7 +136,7 @@ export class ManagedIdentityCredential implements TokenCredential {
   ): Promise<AccessToken> {
     let result: AccessToken | null = null;
 
-    const { span, updatedOptions } = createSpan("ManagedIdentityCredential-getToken", options);
+    const { span, updatedOptions } = createSpan("ManagedIdentityCredential.getToken", options);
 
     try {
       // isEndpointAvailable can be true, false, or null,

sdk/identity/identity/src/credentials/usernamePasswordCredential.browser.ts

@@ -67,7 +67,7 @@ export class UsernamePasswordCredential implements TokenCredential {
     options?: GetTokenOptions
   ): Promise<AccessToken | null> {
     const { span, updatedOptions: newOptions } = createSpan(
-      "UsernamePasswordCredential-getToken",
+      "UsernamePasswordCredential.getToken",
       options
     );
     try {

sdk/identity/identity/src/util/tracing.ts

@@ -9,7 +9,7 @@ import { OperationOptions } from "@azure/core-client";
  * @internal
  */
 export const createSpan = createSpanFunction({
-  packagePrefix: "Azure.Identity",
+  packagePrefix: "",
   namespace: "Microsoft.AAD"
 });
 

sdk/identity/identity/test/public/node/azureApplicationCredential.spec.ts

@@ -61,13 +61,13 @@ describe("AzureApplicationCredential", function() {
       },
       children: [
         {
-          name: "Azure.Identity.ChainedTokenCredential-getToken",
+          name: "ChainedTokenCredential.getToken",
           children: [
             {
-              name: "Azure.Identity.EnvironmentCredential.getToken",
+              name: "EnvironmentCredential.getToken",
               children: [
                 {
-                  name: "Azure.Identity.ClientSecretCredential.getToken",
+                  name: "ClientSecretCredential.getToken",
                   children: []
                 }
               ]

sdk/identity/identity/test/public/node/clientCertificateCredential.spec.ts

@@ -122,7 +122,7 @@ describe("ClientCertificateCredential", function() {
       },
       children: [
         {
-          name: "Azure.Identity.ClientCertificateCredential.getToken",
+          name: "ClientCertificateCredential.getToken",
           children: []
         }
       ]

sdk/identity/identity/test/public/node/clientSecretCredential.spec.ts

@@ -74,7 +74,7 @@ describe("ClientSecretCredential", function() {
       },
       children: [
         {
-          name: "Azure.Identity.ClientSecretCredential.getToken",
+          name: "ClientSecretCredential.getToken",
           children: []
         }
       ]

sdk/identity/identity/test/public/node/deviceCodeCredential.spec.ts

@@ -160,7 +160,7 @@ describe("DeviceCodeCredential", function() {
       },
       children: [
         {
-          name: "Azure.Identity.DeviceCodeCredential.getToken",
+          name: "DeviceCodeCredential.getToken",
           children: []
         }
       ]

sdk/identity/identity/test/public/node/environmentCredential.spec.ts

@@ -122,10 +122,10 @@ describe("EnvironmentCredential", function() {
       },
       children: [
         {
-          name: "Azure.Identity.EnvironmentCredential.getToken",
+          name: "EnvironmentCredential.getToken",
           children: [
             {
-              name: "Azure.Identity.ClientSecretCredential.getToken",
+              name: "ClientSecretCredential.getToken",
               children: []
             }
           ]
@@ -161,10 +161,10 @@ describe("EnvironmentCredential", function() {
       },
       children: [
         {
-          name: "Azure.Identity.EnvironmentCredential.getToken",
+          name: "EnvironmentCredential.getToken",
           children: [
             {
-              name: "Azure.Identity.ClientCertificateCredential.getToken",
+              name: "ClientCertificateCredential.getToken",
               children: []
             }
           ]
@@ -197,10 +197,10 @@ describe("EnvironmentCredential", function() {
       },
       children: [
         {
-          name: "Azure.Identity.EnvironmentCredential.getToken",
+          name: "EnvironmentCredential.getToken",
           children: [
             {
-              name: "Azure.Identity.UsernamePasswordCredential.getToken",
+              name: "UsernamePasswordCredential.getToken",
               children: []
             }
           ]

sdk/identity/identity/test/public/node/usernamePasswordCredential.spec.ts

@@ -84,7 +84,7 @@ describe("UsernamePasswordCredential", function() {
       },
       children: [
         {
-          name: "Azure.Identity.UsernamePasswordCredential.getToken",
+          name: "UsernamePasswordCredential.getToken",
           children: []
         }
       ]