Skip to content

Commit 5634ddc

Browse files
octoperoctoper
authored
fix(remix): Fix issues caused by modifying duplex on Request (clerk#3645)
1 parent d13ef34 commit 5634ddc

2 files changed

Lines changed: 7 additions & 1 deletion

File tree

‎.changeset/tall-hairs-dream.md‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@clerk/remix": patch
3+
---
4+
5+
Fix issues caused by modifying duplex property on Request when it was already defined

‎packages/remix/src/ssr/utils.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,8 @@ export const wrapWithClerkState = (data: any) => {
126126
*/
127127
export const patchRequest = (request: Request) => {
128128
const clonedRequest = request.clone();
129-
if (clonedRequest.method !== 'GET' && clonedRequest.body !== null) {
129+
// If duplex is not set, set it to 'half' to avoid duplex issues with unidici
130+
if (clonedRequest.method !== 'GET' && clonedRequest.body !== null && !('duplex' in clonedRequest)) {
130131
(clonedRequest as unknown as { duplex: 'half' }).duplex = 'half';
131132
}
132133
return clonedRequest;

0 commit comments

Comments
 (0)