-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix websocket protocol for demo site #719
Conversation
@@ -13,6 +13,8 @@ const wsHost = | |||
? 'localhost:4142' | |||
: window.location.host; | |||
|
|||
const wsProtocol = window.location.protocol === 'https:' ? 'wss' : 'ws'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to use this on line 19.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops the file wasn't saved and commited. Nice catch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice.
Signed-off-by: Lim Hoang <[email protected]>
RELEASE.md
Outdated
<!-- | ||
Use the sections below to add notes for the next release. | ||
Please follow the established format: | ||
- Keep each note concise - ideally commit title length | ||
- Use present tense (e.g. 'Add new feature') | ||
- Include the ID number for the related PR (or PRs) in parentheses | ||
--> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should keep this in though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated
Description
The demo site is served through HTTPS so we need
wss
for websocket.Development notes
Lightsail does have a healthcheck for the index. The trouble is if websocket fails the server still returns an HTTP 200. We might need to look at a fix in apollo client to make it fail more gracefully and doesn't blow up the entire app.
QA notes
Checklist
RELEASE.md
file