Skip to content

Commit 62f41e5

Browse files
committed
fix: default to Web stream when +server is enabled
1 parent 76c95b7 commit 62f41e5

2 files changed

Lines changed: 12 additions & 0 deletions

File tree

examples/full/pages/+server.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
import vikeFetch from 'vike/fetch'
2+
3+
export default {
4+
fetch: vikeFetch.fetch,
5+
}

packages/vike-react/src/integration/onRenderHtml.tsx

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -238,6 +238,7 @@ type StreamSetting = {
238238
function resolveStreamSetting(pageContext: PageContextServer): StreamSetting {
239239
const {
240240
stream,
241+
server,
241242
// TO-DO/eventually: remove +streamIsRequired
242243
// - Let's remove it once following last vike-react-{query,apollo} releases using +streamIsRequired can be considered old versions.
243244
// - Last vike-react-query version that uses +streamIsRequired was 0.1.3
@@ -274,5 +275,11 @@ function resolveStreamSetting(pageContext: PageContextServer): StreamSetting {
274275
isType<never>(setting)
275276
throw new Error(`Unexpected +stream value ${setting}`)
276277
})
278+
279+
// If `+server` is defined, we expect a Web Response
280+
if (server && streamSetting.enable && !streamSetting.type) {
281+
streamSetting.type = 'web'
282+
}
283+
277284
return streamSetting
278285
}

0 commit comments

Comments
 (0)