report diagnostics on unsupported auth

packages/http-client-csharp/emitter/src/lib/logger.ts

@@ -1,7 +1,7 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT License. See License.txt in the project root for license information.
 
-import { NoTarget, Program, Tracer } from "@typespec/compiler";
+import { DiagnosticTarget, NoTarget, Program, Tracer } from "@typespec/compiler";
 import { getTracer, reportDiagnostic } from "./lib.js";
 import { LoggerLevel } from "./log-level.js";
 
@@ -63,19 +63,19 @@ export class Logger {
     }
   }
 
-  warn(message: string): void {
+  warn(message: string, target?: DiagnosticTarget | typeof NoTarget): void {
     reportDiagnostic(this.program, {
       code: "general-warning",
       format: { message: message },
-      target: NoTarget,
+      target: target ?? NoTarget,
     });
   }
 
-  error(message: string): void {
+  error(message: string, target?: DiagnosticTarget | typeof NoTarget): void {
     reportDiagnostic(this.program, {
       code: "general-error",
       format: { message: message },
-      target: NoTarget,
+      target: target ?? NoTarget,
     });
   }
 }

packages/http-client-csharp/emitter/src/lib/service-authentication.ts

@@ -47,37 +47,49 @@ function processAuthType(credentialType: SdkCredentialType): InputAuth | undefin
   const scheme = credentialType.scheme;
   switch (scheme.type) {
     case "apiKey":
-      return { ApiKey: { Name: scheme.name } } as InputAuth;
+      if (scheme.in !== "header") {
+        Logger.getInstance().warn(
+          `Only header is supported for ApiKey auth method. ${scheme.in} is not supported.`,
+          credentialType.__raw,
+        );
+        return undefined;
+      }
+      return { ApiKey: { Name: scheme.name, In: scheme.in } } as InputAuth;
     case "oauth2":
       return processOAuth2(scheme);
-    case "http":
-      {
-        const schemeOrApiKeyPrefix = scheme.scheme;
-        switch (schemeOrApiKeyPrefix) {
-          case "basic":
-            Logger.getInstance().warn(
-              `${schemeOrApiKeyPrefix} auth method is currently not supported.`,
-            );
-            return undefined;
-          case "bearer":
-            return {
-              ApiKey: {
-                Name: "Authorization",
-                Prefix: "Bearer",
-              },
-            } as InputAuth;
-          default:
-            return {
-              ApiKey: {
-                Name: "Authorization",
-                Prefix: schemeOrApiKeyPrefix,
-              },
-            } as InputAuth;
-        }
+    case "http": {
+      const schemeOrApiKeyPrefix = scheme.scheme;
+      switch (schemeOrApiKeyPrefix) {
+        case "basic":
+          Logger.getInstance().warn(
+            `${schemeOrApiKeyPrefix} auth method is currently not supported.`,
+            credentialType.__raw,
+          );
+          return undefined;
+        case "bearer":
+          return {
+            ApiKey: {
+              Name: "Authorization",
+              In: "header",
+              Prefix: "Bearer",
+            },
+          };
+        default:
+          return {
+            ApiKey: {
+              Name: "Authorization",
+              In: "header",
+              Prefix: schemeOrApiKeyPrefix,
+            },
+          };
       }
-      break;
+    }
     default:
-      throw new Error(`un-supported authentication scheme ${scheme.type}`);
+      Logger.getInstance().error(
+        `un-supported authentication scheme ${scheme.type}`,
+        credentialType.__raw,
+      );
+      return undefined;
   }
 }
 

packages/http-client-csharp/emitter/src/type/input-api-key-auth.ts

@@ -3,5 +3,8 @@
 
 export interface InputApiKeyAuth {
   Name: string;
+  In: ApiKeyLocation;
   Prefix?: string;
 }
+
+export type ApiKeyLocation = "header"; // | "query" | "cookie"; // we do not support query or cookie yet

packages/http-client-csharp/generator/TestProjects/CadlRanch/http/authentication/api-key/tspCodeModel.json

@@ -168,7 +168,8 @@
   "$id": "18",
   "ApiKey": {
    "$id": "19",
-   "Name": "x-ms-api-key"
+   "Name": "x-ms-api-key",
+   "In": "header"
   }
  }
 }

packages/http-client-csharp/generator/TestProjects/CadlRanch/http/authentication/http/custom/tspCodeModel.json

@@ -169,6 +169,7 @@
   "ApiKey": {
    "$id": "19",
    "Name": "Authorization",
+   "In": "header",
    "Prefix": "SharedAccessKey"
   }
  }

packages/http-client-csharp/generator/TestProjects/CadlRanch/http/authentication/union/tspCodeModel.json

@@ -110,7 +110,8 @@
   "$id": "12",
   "ApiKey": {
    "$id": "13",
-   "Name": "x-ms-api-key"
+   "Name": "x-ms-api-key",
+   "In": "header"
   },
   "OAuth2": {
    "$id": "14",

packages/http-client-csharp/generator/TestProjects/Local/Unbranded-TypeSpec/tspCodeModel.json

@@ -3684,7 +3684,8 @@
   "$id": "348",
   "ApiKey": {
    "$id": "349",
-   "Name": "my-api-key"
+   "Name": "my-api-key",
+   "In": "header"
   }
  }
 }