Skip to content

GitLab

Open
Created by Tom JORQUERA@tjorqueraContributor0 of 6 tasks completed0/6 tasks

Remove proxy

Due to #67 we introduced a proxy in 0ce98228 (tagged with proxy).

This is an ugly workaround, and should be removed when #67 is fixed.

To test that everything work without proxy, do the following in config.json:

  • set client.proxified to false
  • set client.gstreamerURL to wss://:hublot_kaldi-gstreamer_1:80/client/ws/speech?content-type=audio/x-matroska,+rate=(int)48000,+channels=(int) (or whatever is the correct SSL ws URL)
  • set client.reco.host to hublot_recommender_1
  • set client.reco.port to 8080

Everything should work as before, assuming SSL is on (note that every http:// url in live-reco.js is assumed to have been changed to https:// due solving #67).

Does it works? Congrats'!

You can now clean up to remove the proxy:

  • in app.js remove const proxy = ... and proxy.create()
  • in config.json, remove the client.proxified option and the proxy section
  • in config.json ensure client.gstreamerURL, client.reco.host and client.reco.port are set to their correct un-proxified values
  • remove lib/proxy.js, it is no longer useful
  • in live-reco.js you can remove proxifiedConnectionHandler and the test in createConnection, as they should be no longer useful
  • in package.json, you can now remove the sockjs-client, webstomp-client, ws and xmlhttprequest dependencies, assuming they are still not used anywhere else (remember to update yarn.lock)
Edited by Tom JORQUERA