Skip to content

Timezone error for America/Santiago #72

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

Open
joyarzun opened this issue May 28, 2024 · 9 comments
Open

Timezone error for America/Santiago #72

joyarzun opened this issue May 28, 2024 · 9 comments
Labels
bug Something isn't working

Comments

@joyarzun
Copy link

joyarzun commented May 28, 2024

Hello there
I've setup all and it's working but the timezone still appears to be incorrect. I've set America/Santiago and it should show 21:50 but it's showing 1:50. I did change to America/Sao_Paulo and it works. I've changed the NTP server to others but it still continue to showing the same. What else can I do?
I'm using the Mario theme flashed from the web btw

@joyarzun
Copy link
Author

More info:
When I've set the Posix timezone for America/Santiago that is "<-04>4<-03>,M9.1.6/24,M4.1.6/24" the browser does a request like this:
http://192.168.0.xxx/set?manualPosix=%3C-04%3E4%3C-03%3E,M9.1.6/24,M4.1.6/24
so it is doing a parsing of the symbols < and >. Then I did the request with cURL without doing the parsing and it worked:

curl 'http://192.168.0.xxx/set?manualPosix=<-04>4<-03>,M9.1.6/24,M4.1.6/24' \ -X 'POST' \ -H 'Accept: */*' \ -H 'Accept-Language: en-US,en;q=0.9,es-CL;q=0.8,es;q=0.7,pt;q=0.6' \ -H 'Connection: keep-alive' \ -H 'Content-Length: 0' \ -H 'Origin: http://192.168.0.209' \ -H 'Referer: http://192.168.0.209/' \ -H 'User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/125.0.0.0 Safari/537.36' \ -H 'dnt: 1' \ -H 'sec-gpc: 1' \ --insecure

So the UI is failing to support < and > symbols that is Posix standard for some timezones

@jnthas
Copy link
Owner

jnthas commented May 29, 2024

Hi @joyarzun! It seems a bug in the manualPosix, maybe we should force avoiding URL encode. I'm wondering why it didn't work when you set America/Santiago. Have you tried set other NTP server that is not Google? It should work with timezone name.

@jnthas jnthas added the bug Something isn't working label May 29, 2024
@joyarzun
Copy link
Author

joyarzun commented Jun 2, 2024

Hi @joyarzun! It seems a bug in the manualPosix, maybe we should force avoiding URL encode. I'm wondering why it didn't work when you set America/Santiago. Have you tried set other NTP server that is not Google? It should work with timezone name.

Indeed, I've tried with others ntp servers but the result is the same, also I've tried with the eztime lib without success but it's works with America/Sao_Paulo or Pacific/Auckland, so I think we have two issues, first is a bug the the lib (this repo is affected only) and the second is the parsing of Posix timezone

@x0SiN0x
Copy link

x0SiN0x commented Feb 19, 2025

Has there been any progress on this? I loaded the latest version using America/Denver and its not correct

@DocGEEK27
Copy link

DocGEEK27 commented Feb 20, 2025 via email

@x0SiN0x
Copy link

x0SiN0x commented Feb 20, 2025

Finally got mine working using the "Posix Timezone String". I left the timezone alone and just added "MST7MDT,M3.2.0,M11.1.0" to that Posix setting and working great now. Actually boots faster too which was an interesting side effect.

@jnthas
Copy link
Owner

jnthas commented Feb 22, 2025

There's something else with those timezones that I can't replicate here.. I mean, the timezone works fine for me (yes, it works on my machine :D) I just set the timezone to Denver, and it displays correctly.

Image

Image

@jnthas
Copy link
Owner

jnthas commented Feb 22, 2025

Also, please check this issue, it can be the reason
#39

@x0SiN0x
Copy link

x0SiN0x commented Feb 22, 2025

Ahhh, could be that part mentioned about "port 2342 (UDP)", I allow out NTP (standard 123) and various other required traffic but port 2342 is not on that list. I will need to test this once i get back but i bet that is the case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants