-
Notifications
You must be signed in to change notification settings - Fork 1
Websocket altenate server test #2
base: master
Are you sure you want to change the base?
Conversation
Great. Looks good. Thanks ! For the client side webpack build, maybe /docs could be a better choice, as it's the default (and only choice outside master's root) for github page. Not important if we put both client/server in our stack. But maybe some people will still use fxi.io/map-x-tests, which is hosted here. I have to check with @thomaspiller what was the dedicated subdomain for those tests so we could put our the WS_MAPX_URI as an environment variable. As soon as I'm ok with this, I will merge. |
Ok, I didn't realize it was hosted here; so yes we should use /docs for webpack build (let's change that the same time we set the correct WS_MAPX_URI for production). |
Hi @trepmag, Thanks for your work! It's very helpful! Do you think you can 1) extend the duration of the test on the maintainance of the connexion to an undefined time and 2) allow a user to stop it on demand? For instance with a simple button that terminates the ping test. Thanks |
Hi, yes the test can be modified the way you mentioned. Now, I'm wondering if the 'Copy to your clipboard' button should be disabled until the user click the webservice test stop button? If not the user might copy and send an unfinished websocket test... |
I would say you're right. |
…ve some trailing spaces.
…utton to end that test
Hi, I added this infinite websocket ping test to the same PR branch... |
@antobenve Yep, I can't really merge this into master, as it's also the same branch as the live page. We should make both services public first. See with @thomaspiller. Try to make this happen locally, both images are already built. Then make the same in the prod stack. Once it's up and running, I will remove or redirect the github page linked to this repo. |
Hello Fred,
This pull request add a websocket test to an alternate echo server beside the initial one at echo.websocket.org.
The alternate echo server use the docker jmalloc/echo-server image as you suggested.
The javascript test use a variable WS_MAPX_URI to set the correct address which can be differentiated for prod in the webpack.prod.config.js file; then it requires the script to be built as follow:
$ npm run prod
Note, as I don't know the deployment details, the webpack generated dist/main.js is actually part of the repository...