No reviewers
Labels
No labels
Вид/Баг
Вид/Безопасность
Вид/Документация
Вид/Изучение
Вид/Тестирование
Вид/Улучшение
Приоритет
Критический
Приоритет
Высокий
Приоритет
Низкий
Приоритет
Средний
Рассмотрено
Дубликат
Рассмотрено
Не будет исправленно
Рассмотрено
Неверно
Рассмотрено
Подтверждено
Совместимость/Сломанная
Статус
Заблокированно
Статус
Заброшено
Статус
Требуется больше информации
Тег/NarodMon Совместимость
Хорошая первая issue
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: iotishnik/server#14
Loading…
Reference in a new issue
No description provided.
Delete branch "device_managment"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
3e684d093f
to956905e6e8
WIP: feat: begin work on #7 and #10to feat: begin work on #7 and #10Сейчас от сокет сервера только файл заглушка. Пока только сделал разделение кода заливки сущностей в отдельный файл. Нужно это залить сейчас чтобы я смог наконец-то настроить форматирование, пока не поздно.
И ещё сделал заливку команд в базу. Чтение будет потом в сокет сервере.
@ -86,0 +110,4 @@
// TODO: Добавить гистерезис
// Отчёт совместимости: отсутствует
if device_time > now {
Можно сделать:
Можно и наверное так канонично делать в расте, но это не читабельно. Уж извини, но этот вариант мне не нравится в данном случае.
А точно
let mut
плохо для производительности не будет?Если считаешь более читабельным – окей, пусть так будет. Мне наоборот кажется, что странно выглядит.
@ -111,6 +111,10 @@ pub async fn parse_nm_qs_format(input: &str) -> Result<NMDeviceDataPacket, QSPar
lon: parse_decimal_if_exists(&mut parsed, "lon")?,
alt: parse_decimal_if_exists(&mut parsed, "alt")?,
time: parse_epoch_if_exists(&mut parsed, "time")?,
// TODO: Выяснить можно ли передавать команды по QS и можно ли их отличить от
Скорее всего нет, или как минимум никто в здравом уме не будет через кверистринг слать список команд, я боюсь представить чем там кодируется список (типа
&cmd1=...&cmd2=...
?).В доках нету => а как юзеры-то об этой фиче узнают, даже если она есть?
Я это добавил так, на будущее. Вдруг там каким-то магическим образом можно.
Да там в доках большинство не написано, мб где-то так делают на проде. Хз, я и не такое видел в коде некоторых библиотек работы с narodmon.
А, весело, получается у разработчиков и админов выпытывают незадокументированные апи либо сами перебирают варианты)
LGTM