export client metadata domain from config to ensure consistency

Author: essential-randomness

astro-authproto/src/config-module.d.ts

@@ -25,6 +25,7 @@ declare module "fujocoded:authproto/config" {
   export const driverName: string;
   export const redirectAfterLogin: string;
   export const redirectAfterLogout: string;
+  export const clientMetadataDomain: string;
 }
 
 declare module "fujocoded:authproto/stores" {

astro-authproto/src/lib/auth.ts

@@ -8,7 +8,7 @@ import { DidResolver } from "@atproto/identity";
 import {
   scopes,
   applicationName,
-  externalDomain,
+  clientMetadataDomain,
 } from "fujocoded:authproto/config";
 
 // We import the stores from the virtual module "fujocoded:authproto/stores"
@@ -77,9 +77,7 @@ const createClient = async (domain: string) => {
   });
 };
 
-const DOMAIN =
-  process.env.AUTHPROTO_EXTERNAL_DOMAIN ?? externalDomain ?? "http://127.0.0.1:4321/";
-export const oauthClient = await createClient(DOMAIN);
+export const oauthClient = await createClient(clientMetadataDomain);
 
 const IDENTITY_RESOLVER = new DidResolver({});
 export const didToHandle = async (did: string) => {

astro-authproto/src/lib/config.ts

@@ -128,5 +128,6 @@ export const getConfig = ({ options, isDev }: { options: ConfigOptions, isDev: b
       options.redirects?.afterLogout ?? "/"
     )};
     export const externalDomain = ${JSON.stringify(externalDomain)};
+    export const clientMetadataDomain = process.env.AUTHPROTO_EXTERNAL_DOMAIN ?? ${JSON.stringify(externalDomain)} ?? "${options.applicationDomain}";
     `;
 };

astro-authproto/src/routes/oauth-client-metadata.json.ts

@@ -1,10 +1,10 @@
 import type { APIRoute } from "astro";
-import { applicationDomain } from "fujocoded:authproto/config";
+import { clientMetadataDomain } from "fujocoded:authproto/config";
 import { createClientMetadata } from "../lib/auth.js";
 
 export const GET: APIRoute = async ({}) => {
   return new Response(
-    JSON.stringify(createClientMetadata(applicationDomain)),
+    JSON.stringify(createClientMetadata(clientMetadataDomain)),
     {
       status: 200,
       headers: {