remove site requirement from netlify adapter (#3041)

* remove site requirement from netlify adapter

* update readme

packages/integrations/netlify/src/README.md

@@ -10,10 +10,6 @@ import netlify from '@astrojs/netlify/functions';
 
 export default defineConfig({
 	adapter: netlify(),
-  // Where your Netlify app will be deployed.
-  // Feel free to use a local URL (i.e. http://localhost:8080)
-  // to test local builds via the netlify CLI
-  site: 'https://my-production-url.netlify.app',
 });
 ```
 

packages/integrations/netlify/src/index.ts

@@ -1,12 +1,12 @@
 import type { AstroAdapter, AstroIntegration, AstroConfig } from 'astro';
 import fs from 'fs';
 
-export function getAdapter(site: string | undefined): AstroAdapter {
+export function getAdapter(): AstroAdapter {
 	return {
 		name: '@astrojs/netlify',
 		serverEntrypoint: '@astrojs/netlify/netlify-functions.js',
 		exports: ['handler'],
-		args: { site },
+		args: { },
 	};
 }
 
@@ -28,15 +28,7 @@ function netlifyFunctions({ dist }: NetlifyFunctionsOptions = {}): AstroIntegrat
 				}
 			},
 			'astro:config:done': ({ config, setAdapter }) => {
-				let site = null;
+				setAdapter(getAdapter());
-				try {
-					site = new URL(config.base, config.site);
-				} catch {
-					throw new Error(
-						'The Netlify adapter requires a deployment URL. Ensure a "site" is specified in your astro.config. If you provided a "base" in your astro.config, ensure it is a valid path.'
-					);
-				}
-				setAdapter(getAdapter(site.toString()));
 				_config = config;
 			},
 			'astro:build:start': async ({ buildConfig }) => {

packages/integrations/netlify/src/netlify-functions.ts

@@ -7,20 +7,24 @@ polyfill(globalThis, {
 	exclude: 'window document',
 });
 
-interface Args {
+interface Args {}
-	site?: string;
-}
 
 export const createExports = (manifest: SSRManifest, args: Args) => {
 	const app = new App(manifest);
-	const site = new URL(args.site ?? `https://netlify.com`);
 
 	const handler: Handler = async (event) => {
-		const headers = new Headers(event.headers as any);
+		const { httpMethod, headers, rawUrl, body: requestBody, isBase64Encoded } = event;
-		const request = new Request(new URL(event.path, site).toString(), {
+		const init: RequestInit = {
-			method: event.httpMethod,
+			method: httpMethod,
-			headers,
+			headers: new Headers(headers as any),
-		});
+		};
+		// Attach the event body the the request, with proper encoding.
+		if (httpMethod !== 'GET' && httpMethod !== 'HEAD') {
+			const encoding = isBase64Encoded ? 'base64' : 'utf-8';
+			init.body =
+				typeof requestBody === 'string' ? Buffer.from(requestBody, encoding) : requestBody;
+		}
+		const request = new Request(rawUrl, init);
 
 		if (!app.match(request)) {
 			return {
@@ -30,12 +34,12 @@ export const createExports = (manifest: SSRManifest, args: Args) => {
 		}
 
 		const response = await app.render(request);
-		const body = await response.text();
+		const responseBody = await response.text();
 
 		return {
 			statusCode: 200,
 			headers: Object.fromEntries(response.headers.entries()),
-			body,
+			body: responseBody,
 		};
 	};