Исправление сериализации Epoch #13

nm17 · 2024-05-31T23:59:53+03:00

nm17 commented

2024-05-31 23:59:53 +03:00

Явно забыл добавить её ранее

nm17 added 1 commit 2024-05-31 23:59:54 +03:00

fix: add proper epoch serialization 6090b2b47f

requested review from DarkCat09

2024-06-01 00:00:52 +03:00

DarkCat09 reviewed 2024-06-01 09:41:50 +03:00

src/utils/hifitime_serde.rs Outdated

					
				@ -0,0 +54,4 @@

				                    E: de::Error,

				            {

				                return Ok(Epoch::from_unix_seconds(

				                    value.parse().map_err(de::Error::custom)?

DarkCat09 commented

2024-06-01 09:41:35 +03:00

API народмона также разрешает отправлять юникс-время в 16-ричном виде. Предполагаю, растовый parse() по умолчанию парсит только decimal и выдаст ошибку на hex. Верно?

API народмона также разрешает отправлять юникс-время в 16-ричном виде. Предполагаю, растовый `parse()` по умолчанию парсит только decimal и выдаст ошибку на hex. Верно?

nm17 commented

2024-06-01 09:55:09 +03:00

API народмона также разрешает отправлять юникс-время в 16-ричном виде.

Чаво? Это где написано? Впервые слышу.

> API народмона также разрешает отправлять юникс-время в 16-ричном виде. Чаво? Это где написано? Впервые слышу.

DarkCat09 commented

2024-06-01 10:03:16 +03:00

А вот)

Screenshot_20240601-110112_Mull.png

232 KiB

nm17 commented

2024-06-01 10:06:45 +03:00

А, хорошо подметил. Тогда доделаю как смогу.

👍 1

DarkCat09 commented

2024-06-01 10:09:11 +03:00

Предполагаю, что пользуются этим не один-два человека, экономить трафик все хотят. Реализацию я вижу вида:

попытаться распарсить 10-ричное число
получилось – проверить, не прошедший ли timestamp указан, и если да – проверить то же самое с 16-ричной конвертацией, скорее всего так получится правильное время
не получилось – парсим как hex

Честно не понимаю, как они это на своём сервере на баш-скриптах обрабатывают. Гениальный API.

Предполагаю, что пользуются этим не один-два человека, экономить трафик все хотят. Реализацию я вижу вида: 1. попытаться распарсить 10-ричное число 2. получилось – проверить, не прошедший ли timestamp указан, и если да – проверить то же самое с 16-ричной конвертацией, скорее всего так получится правильное время 3. не получилось – парсим как hex Честно не понимаю, как они это на своём сервере на баш-скриптах обрабатывают. Гениальный API.

👍 1

DarkCat09 commented

2024-06-01 11:05:56 +03:00

[То же самое написал в тг-чате]

Ещё можно, знаешь, смотреть по префиксу таймстемпа.

Например, сейчас префикс 1717 в десятеричном и 0x665 в хексе.
1717228768
0x665ad4e0

Через N секунд обновляем префиксы, если нужно.

Когда приходит time от девайса, пробуем распарсить как десятиричное число, и если получается – то сравниваем по префиксам. Иначе пробуем парсить как хекс.

[То же самое написал в тг-чате] Ещё можно, знаешь, смотреть по префиксу таймстемпа. Например, сейчас префикс 1717 в десятеричном и 0x665 в хексе. 1717228768 0x665ad4e0 Через N секунд обновляем префиксы, если нужно. Когда приходит time от девайса, пробуем распарсить как десятиричное число, и если получается – то сравниваем по префиксам. Иначе пробуем парсить как хекс.

nm17 commented

2024-06-01 11:59:20 +03:00

Мне кажется легче всего отличить по длине строки. 8 символов => значит hex.

DarkCat09 commented

2024-06-01 12:27:32 +03:00

Но ведь когда-то будет не 8 символов, а больше)
Обновлять надо. Похоже на мой вариант с префиксами, кстати. Да, по длине проще.

Но ведь когда-то будет не 8 символов, а больше) Обновлять надо. Похоже на мой вариант с префиксами, кстати. Да, по длине проще.

nm17 commented

2024-06-01 12:50:17 +03:00

Но ведь когда-то будет не 8 символов, а больше)

Это произойдёт 7 Февраля 2106г. в 06:28:15 UTC, так что сойдёт.

> Но ведь когда-то будет не 8 символов, а больше) Это произойдёт 7 Февраля 2106г. в 06:28:15 UTC, так что сойдёт.

nm17 commented

2024-06-01 19:58:14 +03:00

Заметка: байты эти в BE, а не LE

DarkCat09 marked this conversation as resolved

DarkCat09 reviewed 2024-06-01 20:16:54 +03:00

src/utils/hifitime_serde.rs Outdated

					
				@ -0,0 +35,4 @@

				            fn expecting(&self, formatter: &mut Formatter) -> fmt::Result {

				                formatter.write_str(

				                    "a (string, representing a timestamp in number or hex form) or a number",

DarkCat09 commented

2024-06-01 20:16:36 +03:00

string representing a timestamp in a decimal or hex number

`string representing a timestamp in a decimal or hex number`

nm17 commented

2024-06-01 20:23:53 +03:00

string representing a timestamp in a decimal or hex number

Не, совсем не звучит. "Строка, представляющая из себя таймстамп в цифре(?) или шестнадцатеричное число". Сейчас подправлю чтобы точно не было сомнений.

> `string representing a timestamp in a decimal or hex number` Не, совсем не звучит. "Строка, представляющая из себя таймстамп в цифре(?) или шестнадцатеричное число". Сейчас подправлю чтобы точно не было сомнений.

DarkCat09 marked this conversation as resolved

nm17 force-pushed fix/hifitime-serde from 11c830c3f3 to 9edfcb3af7

2024-06-01 20:26:23 +03:00

Compare

DarkCat09 approved these changes 2024-06-01 21:25:22 +03:00

DarkCat09 merged commit c9e81b942e into master

2024-06-01 21:25:50 +03:00

DarkCat09 referenced this pull request from a commit

2024-06-01 21:25:51 +03:00

Merge pull request 'Исправление сериализации Epoch' (#13) from fix/hifitime-serde into master

DarkCat09 deleted branch fix/hifitime-serde

2024-06-01 21:36:51 +03:00