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

Проблемы времени в бинарных версиях #80

Open
kostya-main opened this issue Dec 30, 2024 · 7 comments
Open
Labels
v0.0.5 Функции которые нужно сделать до этой версии

Comments

@kostya-main
Copy link

В последних версиях @yao-pkg/pkg есть подозрение что используют версии node с отключенной интернационализацией.

Проблема проявляется в том что время в логах не корректное, так же пришлось переписовать формат лолгов которые зависели от формата времени и на Fastify ошибки по загрузки релизов (по моему)

@kostya-main kostya-main converted this from a draft issue Dec 30, 2024
@kostya-main kostya-main added the v0.0.5 Функции которые нужно сделать до этой версии label Dec 30, 2024
@kostya-main
Copy link
Author

yao-pkg/pkg#101
Данная issue подтверждает догадки.
https://github.com/yao-pkg/pkg-fetch?tab=readme-ov-file#small-icu
Так же они сами заявляют что перешли на small-icu где не содержатся информация о регионе RU.

Похоже нужно переходить на SEA.

@kostya-main
Copy link
Author

При сборке с аргументом --sea загрузка node идёт с офф сайта (там лежат сборки full-icu). Так что инструмент можно не менять. НО время всё равно идёт по UTC 0.
Оказалась что при new Data().toISOString() UTC 0 а при new Data().toString() у нас уже местное время.

@kostya-main
Copy link
Author

Заключение: можно использовать small-icu если со стороны сетевых функций не будет проблем. Нужно опять переписать LogHelper с использованием другова парсера.

(Для интереса размер файла small-icu 57,6 МБ а full-icu 91,4 МБ windows)

@JoCat JoCat moved this from В очереди на выполнение to В работе in AuroraLauncher Функционал Jan 1, 2025
@JoCat
Copy link
Member

JoCat commented Jan 1, 2025

Точно, момент с часовыми поясами я забыл учесть))

@JoCat
Copy link
Member

JoCat commented Jan 1, 2025

Похоже что toLocaleString работает корректно

image
image

@kostya-main
Copy link
Author

Image
Проблема всё ещё присутствует. Всё таки переходим на full-icu?

@kostya-main
Copy link
Author

Полная переустановка исходников всё решила.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v0.0.5 Функции которые нужно сделать до этой версии
Projects
Status: Тестирование
Development

No branches or pull requests

2 participants