@ant32 right, so maybe this check should be in syncTime() function

@uzi18 I'm not sure how that would work. From what I can see the other places where syncTime is used only the time is important and other places where syncDate is called only the date is important.

Actually it is also used in the begin() function which could result in a race condition there possibly? It doesn't look to me like something that needs an exact date/time. @fpistm might know if it racing there is an issue.

Hi @ant32

Thanks for the issue and PR.
I guess the begin should also be updated as well and order also like in getEpoch to be aligned.

@fpistm added the check to the begin function and changed order to match

Thanks @ant32t
It is merged.