Merge pull request #244 from cm-dyoshikawa/fix-options-headers

jshemas · web-flow · commit 74447edaf1e9 · 2024-08-28T22:34:57.000-04:00
Always use default settings for headers
diff --git a/lib/request.ts b/lib/request.ts
@@ -52,8 +52,8 @@ export default async function requestAndResultsFormatter(options: OpenGraphScrap
       options.url ?? '',
       {
         signal: AbortSignal.timeout((options.timeout ?? 10) * 1000),
-        headers: { Origin: options.url ?? '', Accept: 'text/html' },
         ...options.fetchOptions,
+        headers: { Origin: options.url ?? '', Accept: 'text/html', ...options.fetchOptions?.headers },
       },
     );
 

Original file line number	Diff line number	Diff line change
`@@ -52,8 +52,8 @@ export default async function requestAndResultsFormatter(options: OpenGraphScrap`
`52`	`52`	`options.url ?? '',`
`53`	`53`	`{`
`54`	`54`	`signal: AbortSignal.timeout((options.timeout ?? 10) * 1000),`
`55`		`- headers: { Origin: options.url ?? '', Accept: 'text/html' },`
`56`	`55`	`...options.fetchOptions,`
	`56`	`+ headers: { Origin: options.url ?? '', Accept: 'text/html', ...options.fetchOptions?.headers },`
`57`	`57`	`},`
`58`	`58`	`);`
`59`	`59`