Skip to content
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

NTRIP Server restarted after going from Base to Rover #774

Closed
PaulZC opened this issue Jun 7, 2024 · 3 comments
Closed

NTRIP Server restarted after going from Base to Rover #774

PaulZC opened this issue Jun 7, 2024 · 3 comments

Comments

@PaulZC
Copy link
Contributor

PaulZC commented Jun 7, 2024

When in Rover mode, it should not be possible to start the connections to the NTRIP Servers. But I see the server being started when in Rover mode - after having been in Base.

Provide WiFi credentials.
Provide server details for one (or two) NTRIP servers.
Start the RTK in Temporary Base mode.
Allow the survey-in to complete. Firmware begins sending RTCM data to the servers.
Change to Rover mode.
The NTRIP server connections should be stopped, but I see them being periodically restarted - leading to WiFi badness when going into WiFi Config mode.

image

@PaulZC PaulZC changed the title NTRIP Server perpetually restarted when going from Base to Rover NTRIP Server restarted after going from Base to Rover Jun 7, 2024
@PaulZC
Copy link
Contributor Author

PaulZC commented Jun 7, 2024

After a lot of digging and head-scratching, I eventually traced this to ntripServerRestart being called - instead of ntripServerStop. I believe calling ntripServerStop is the correct thing to do. ntripServerRestart actually turns the server on if the connection limit has not been reached... The same is true for ntripClientStop vs. ntripClientRestart.

@nseidle @LeeLeahy2 : please review this commit and let me know if you agree with the solution : af73bf3

@nseidle
Copy link
Member

nseidle commented Jun 10, 2024

Looks good to me.

@nseidle
Copy link
Member

nseidle commented Jul 22, 2024

Fixed in v4.1.

@nseidle nseidle closed this as completed Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants