Skip to content

Commit 2adcf4a

Browse files
authored
fix: headers merge to support all NitroFetchOptions header types
Refactor header handling in fetch request to use Headers object.
1 parent e29ae1f commit 2adcf4a

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

server/utils/github.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,13 @@ export async function fetchGitHubWithRetries<T>(
1919

2020
for (let attempt = 0; attempt < maxAttempts; attempt += 1) {
2121
try {
22+
const headers = new Headers(defaultHeaders)
23+
for (const [key, value] of new Headers(fetchOptions.headers)) {
24+
headers.set(key, value)
25+
}
2226
const response = await $fetch.raw(url, {
2327
...fetchOptions,
24-
headers: {
25-
...defaultHeaders,
26-
...fetchOptions.headers,
27-
},
28+
headers,
2829
})
2930

3031
if (response.status === 200) {

0 commit comments

Comments
 (0)