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

NTP Timeout Variable in web interface #1628

Closed
destring opened this issue Mar 11, 2019 · 2 comments
Closed

NTP Timeout Variable in web interface #1628

destring opened this issue Mar 11, 2019 · 2 comments
Labels
enhancement New feature or request ntp

Comments

@destring
Copy link

we are having issue with NTP sync and it looks like that request is getting timeout because of 1 second timeout set by default in binaries.

I wanted to increase it but did not find any option... I think it would be handy to have this option in web interface because for some implementations where internet connection is not fast enough and remote ntp server is being used, users would want to change this...

I have also tried to set the variable using debug console but do not know which variable it actually is... I tried to get values for "NTP_TIMEOUT" and "ntpTimeout" but I think it is not the correct variable and looks like it is hardcoded in the binaries... I know it can be changed in the source but I think making it changeable from the debug console or web interface will be helpful.

@destring destring added the enhancement New feature or request label Mar 11, 2019
@mcspr mcspr added the ntp label Mar 11, 2019
@mcspr
Copy link
Collaborator

mcspr commented Mar 25, 2019

#1642 adds ntpTimeout setting
Will need to fixup some web issues before exposing it there though

@mcspr
Copy link
Collaborator

mcspr commented Nov 13, 2020

#2132 replaces the underlying NTP client with the one from the LWIP, timeout & retries are handled via LWIP internally.
NTP_TIMEOUT / ntpTimeout are no longer used

@mcspr mcspr closed this as completed Nov 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ntp
Projects
None yet
Development

No branches or pull requests

2 participants