Skip to content

Fix vite development for desktop websocket endpoint#23815

Merged
ibeckermayer merged 2 commits intomasterfrom
isaiah/fix-vite-dev-ws
Mar 30, 2023
Merged

Fix vite development for desktop websocket endpoint#23815
ibeckermayer merged 2 commits intomasterfrom
isaiah/fix-vite-dev-ws

Conversation

@ibeckermayer
Copy link
Copy Markdown
Contributor

When changeOrigin is set to true, the origin of the request is changed to localhost:3000, which upsets Gorilla websocket's default origin check which is expecting the origin localhost:3080 (the origin the server is running from).

Also includes a minor correction in the README.

Copy link
Copy Markdown
Member

@ryanclark ryanclark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cheers for this

Copy link
Copy Markdown
Contributor

@hatched hatched left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@ibeckermayer ibeckermayer added this pull request to the merge queue Mar 30, 2023
Merged via the queue into master with commit f805722 Mar 30, 2023
@ibeckermayer ibeckermayer deleted the isaiah/fix-vite-dev-ws branch March 30, 2023 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants