search

nightscout / cgm-remote-monitor

nightscout web monitor
GNU Affero General Public License v3.0
2.43k stars 71.81k forks source link

No connection from AAPS since socket.io Update #7874

Closed philipgo closed 1 year ago

philipgo commented 1 year ago

Describe the bug Since commit https://github.com/nightscout/cgm-remote-monitor/commit/6ad4ef18140d33936bf8795c628c698b90ced02c AndroidAPS (dev built Jan 31) cannot connect to my Nightscout instance. In the NSCL tab AndroidAPS shows Error: io.socket.engineio.client.EngineIOException xhr poll error

To Reproduce

  1. Run AndroidAPS dev
  2. Connect to a Nightscout instance running dev including the above mentioned commit
  3. Try syncing your treatments etc with your Nightscout instance

Expected behavior Connection without error

Your setup information

Additional context My nightscout setup is a self hosted docker installation. But that probably does not play a role in this context.

viq commented 1 year ago

Just out of curiousity, are you using the official docker image, or did you build your own with a newer node version?

I'm seeing the same issue on a self hosted version, but I blamed the fact that where I'm running it I have only node 18 available.

philipgo commented 1 year ago

@viq I use this docker image: https://hub.docker.com/r/nightscout/cgm-remote-monitor

If I interpret it correctly, it uses node version 14.15.3

viq commented 1 year ago

So the official one. Yeah, it is 14.15.3. Thank you.

MilosKozak commented 1 year ago

AAPS master + dev needs NS master AAPS ns15 branch needs NS dev

viq commented 1 year ago

Thank you for that information. Skimming through https://github.com/nightscout/AndroidAPS/pull/2364 it seems like most changes touch only the nightscout and xdrip communications, and otherwise running this version should be not too different from running master (i.e. 3.1.0.3), do I understand it correctly?

philipgo commented 1 year ago

AAPS master + dev needs NS master AAPS ns15 branch needs NS dev

Thank you, now it works.