first pass at token auth, rename baseUrl to host to match CLI

oxidecomputer · Jul 21, 2023 · 59f4a79 · 59f4a79
1 parent 8ffcc89
commit 59f4a79
Show file tree

Hide file tree

Showing 2 changed files with 36 additions and 28 deletions.
diff --git a/client/http-client.ts b/client/http-client.ts
@@ -15,17 +15,20 @@ export interface FullRequestParams extends Omit<RequestInit, "body"> {
   path: string;
   query?: QueryParamsType;
   body?: unknown;
-  baseUrl?: string;
+  host?: string;
 }
 
 export type RequestParams = Omit<
   FullRequestParams,
   "body" | "method" | "query" | "path"
 >;
 
+type BaseApiParams = Omit<RequestParams, "host" | "signal">;
+
 export interface ApiConfig {
-  baseUrl?: string;
-  baseApiParams?: Omit<RequestParams, "baseUrl" | "signal">;
+  host?: string;
+  token?: string;
+  baseApiParams?: BaseApiParams;
 }
 
 /** Success responses from the API */
@@ -136,17 +139,21 @@ export async function handleResponse<Data>(
 }
 
 export class HttpClient {
-  public baseUrl = "";
-
-  private baseApiParams: RequestParams = {
+  public host?: string;
+  public token?: string;
+  private baseApiParams: BaseApiParams = {
     credentials: "same-origin",
-    headers: {},
+    headers: {
+      "Content-Type": "application/json",
+    },
     redirect: "follow",
     referrerPolicy: "no-referrer",
   };
 
   constructor(apiConfig: ApiConfig = {}) {
-    Object.assign(this, apiConfig);
+    this.host = apiConfig.host;
+    this.token = apiConfig.token;
+    this.baseApiParams = apiConfig.baseApiParams || this.baseApiParams;
   }
 
   private mergeRequestParams(params: RequestParams): RequestParams {
@@ -155,6 +162,7 @@ export class HttpClient {
       ...params,
       headers: {
         ...this.baseApiParams.headers,
+        ...(this.token ? { Authorization: `Bearer ${this.token}` } : {}),
         ...params.headers,
       },
     };
@@ -164,19 +172,15 @@ export class HttpClient {
     body,
     path,
     query,
-    baseUrl,
+    host,
     ...params
   }: FullRequestParams): Promise<ApiResult<Data>> => {
     const requestParams = this.mergeRequestParams(params);
 
-    const url = (baseUrl || this.baseUrl || "") + path + toQueryString(query);
+    const url = (host || this.host || "") + path + toQueryString(query);
 
     const response = await fetch(url, {
       ...requestParams,
-      headers: {
-        "Content-Type": "application/json",
-        ...requestParams.headers,
-      },
       body: JSON.stringify(snakeify(body), replacer),
     });
 

diff --git a/static/http-client.ts b/static/http-client.ts
@@ -15,17 +15,20 @@ export interface FullRequestParams extends Omit<RequestInit, "body"> {
   path: string;
   query?: QueryParamsType;
   body?: unknown;
-  baseUrl?: string;
+  host?: string;
 }
 
 export type RequestParams = Omit<
   FullRequestParams,
   "body" | "method" | "query" | "path"
 >;
 
+type BaseApiParams = Omit<RequestParams, "host" | "signal">;
+
 export interface ApiConfig {
-  baseUrl?: string;
-  baseApiParams?: Omit<RequestParams, "baseUrl" | "signal">;
+  host?: string;
+  token?: string;
+  baseApiParams?: BaseApiParams;
 }
 
 /** Success responses from the API */
@@ -136,17 +139,21 @@ export async function handleResponse<Data>(
 }
 
 export class HttpClient {
-  public baseUrl = "";
-
-  private baseApiParams: RequestParams = {
+  public host?: string;
+  public token?: string;
+  private baseApiParams: BaseApiParams = {
     credentials: "same-origin",
-    headers: {},
+    headers: {
+      "Content-Type": "application/json",
+    },
     redirect: "follow",
     referrerPolicy: "no-referrer",
   };
 
   constructor(apiConfig: ApiConfig = {}) {
-    Object.assign(this, apiConfig);
+    this.host = apiConfig.host;
+    this.token = apiConfig.token;
+    this.baseApiParams = apiConfig.baseApiParams || this.baseApiParams;
   }
 
   private mergeRequestParams(params: RequestParams): RequestParams {
@@ -155,6 +162,7 @@ export class HttpClient {
       ...params,
       headers: {
         ...this.baseApiParams.headers,
+        ...(this.token ? { Authorization: `Bearer ${this.token}` } : {}),
         ...params.headers,
       },
     };
@@ -164,19 +172,15 @@ export class HttpClient {
     body,
     path,
     query,
-    baseUrl,
+    host,
     ...params
   }: FullRequestParams): Promise<ApiResult<Data>> => {
     const requestParams = this.mergeRequestParams(params);
 
-    const url = (baseUrl || this.baseUrl || "") + path + toQueryString(query);
+    const url = (host || this.host || "") + path + toQueryString(query);
 
     const response = await fetch(url, {
       ...requestParams,
-      headers: {
-        "Content-Type": "application/json",
-        ...requestParams.headers,
-      },
       body: JSON.stringify(snakeify(body), replacer),
     });