Add config option for WebSockets port #51
Conversation
| ws = new WebSocket(`${ proto }://${ window.document.location.host }`); | ||
| var port = 80; | ||
| if (config.wsPort > 0) { | ||
| port = config.wsPort; |
There was a problem hiding this comment.
Mozaïk code base uses 4 spaces indent
|
@arowla, sorry for the late review and thank you for your contribution! |
Add config option for WebSockets port
|
@arowla available in v1.0.10 (host config too) |
|
@arowla, I'm about to add a references section on http://mozaik.rocks/; being a side project (in other words, working on it on my spare time :)), it's hard to craft and promote the tool, so I was wondering if I can add a reference to https://18f.gsa.gov/ and if you could say a few words on your experience with it. I hope you won't find this message too annoying, and sorry to use a comment on github for that. |
|
@plouc Sorry to get back to this so late. We experimented with Mozaik, but I'm not sure whether that project ever got wings. I'm now no longer at 18F, so I'm afraid I can't comment beyond that. |
At 18F, we were recently playing around with deploying the mozaik-demo on https://cloud.gov. Our hosting environment requires that WebSockets be served on a port other than 80. This PR adds an option to configure that port.