iotishnik/server
2
2
Fork 0
Code Issues 11 Pull requests Projects Releases Activity Actions

feat: begin work on #7 and #10 #14

Manually merged
nm17 merged 1 commit from device_managment into master 2024-06-01 23:24:31 +03:00
Conversation 2 Commits 1 Files changed 9 +845 -677
nm17 commented 2024-06-01 00:25:49 +03:00
Owner
Copy link
No description provided.
nm17 added 1 commit 2024-06-01 00:25:49 +03:00
nm17 force-pushed device_managment from 3e684d093f to 956905e6e8 2024-06-01 22:31:35 +03:00 Compare
nm17 changed title from WIP: feat: begin work on #7 and #10 to feat: begin work on #7 and #10 2024-06-01 22:31:44 +03:00
nm17 commented 2024-06-01 22:33:09 +03:00
Author
Owner
Copy link

Сейчас от сокет сервера только файл заглушка. Пока только сделал разделение кода заливки сущностей в отдельный файл. Нужно это залить сейчас чтобы я смог наконец-то настроить форматирование, пока не поздно.

Сейчас от сокет сервера только файл заглушка. Пока только сделал разделение кода заливки сущностей в отдельный файл. Нужно это залить сейчас чтобы я смог наконец-то настроить форматирование, пока не поздно.
nm17 requested review from DarkCat09 2024-06-01 22:33:25 +03:00
nm17 commented 2024-06-01 22:34:18 +03:00
Author
Owner
Copy link

И ещё сделал заливку команд в базу. Чтение будет потом в сокет сервере.

И ещё сделал заливку команд в базу. Чтение будет потом в сокет сервере.
DarkCat09 reviewed 2024-06-01 22:50:56 +03:00
src/ingest_protocol/packet_types.rs
@ -86,0 +110,4 @@
// TODO: Добавить гистерезис
// Отчёт совместимости: отсутствует
if device_time > now {
DarkCat09 commented 2024-06-01 22:42:12 +03:00
Owner
Copy link

Можно сделать:

let device_time = self.time
  .map(|t| if t > now {
    now
  } else {
    t
  })
  .unwrap_or(now)
Можно сделать: ```rust let device_time = self.time .map(|t| if t > now { now } else { t }) .unwrap_or(now) ```
nm17 commented 2024-06-01 22:58:02 +03:00
Author
Owner
Copy link

Можно и наверное так канонично делать в расте, но это не читабельно. Уж извини, но этот вариант мне не нравится в данном случае.

Можно и наверное так канонично делать в расте, но это не читабельно. Уж извини, но этот вариант мне не нравится в данном случае.
DarkCat09 commented 2024-06-01 23:00:27 +03:00
Owner
Copy link

А точно let mut плохо для производительности не будет?

Если считаешь более читабельным – окей, пусть так будет. Мне наоборот кажется, что странно выглядит.

А точно `let mut` плохо для производительности не будет? Если считаешь более читабельным – окей, пусть так будет. Мне наоборот кажется, что странно выглядит.
DarkCat09 marked this conversation as resolved
src/web_server/old_device_sensor_api/qs_parser.rs
@ -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 и можно ли их отличить от
DarkCat09 commented 2024-06-01 22:49:26 +03:00
Owner
Copy link

Скорее всего нет, или как минимум никто в здравом уме не будет через кверистринг слать список команд, я боюсь представить чем там кодируется список (типа &cmd1=...&cmd2=... ?).

В доках нету => а как юзеры-то об этой фиче узнают, даже если она есть?

Скорее всего нет, или как минимум никто в здравом уме не будет через кверистринг слать список команд, я боюсь представить чем там кодируется список (типа `&cmd1=...&cmd2=...` ?). В доках нету => а как юзеры-то об этой фиче узнают, даже если она есть?
nm17 commented 2024-06-01 22:54:49 +03:00
Author
Owner
Copy link

Я это добавил так, на будущее. Вдруг там каким-то магическим образом можно.

В доках нету => а как юзеры-то об этой фиче узнают, даже если она есть?

Да там в доках большинство не написано, мб где-то так делают на проде. Хз, я и не такое видел в коде некоторых библиотек работы с narodmon.

Я это добавил так, на будущее. Вдруг там каким-то магическим образом можно. > В доках нету => а как юзеры-то об этой фиче узнают, даже если она есть? Да там в доках большинство не написано, мб где-то так делают на проде. Хз, я и не такое видел в коде некоторых библиотек работы с narodmon.
DarkCat09 commented 2024-06-01 22:58:26 +03:00
Owner
Copy link

А, весело, получается у разработчиков и админов выпытывают незадокументированные апи либо сами перебирают варианты)

А, весело, получается у разработчиков и админов выпытывают незадокументированные апи либо сами перебирают варианты)
DarkCat09 marked this conversation as resolved
DarkCat09 approved these changes 2024-06-01 23:03:16 +03:00
DarkCat09 left a comment
Owner
Copy link

LGTM

LGTM
nm17 manually merged commit 9617f5ffbe into master 2024-06-01 23:24:31 +03:00
nm17 deleted branch device_managment 2024-06-01 23:41:29 +03:00
Sign in to join this conversation.
Reviewers
No reviewers
DarkCat09
Labels
Clear labels   
Вид/Баг

Something is not working

  
Вид/Безопасность

This is security issue

  
Вид/Документация

Documentation changes

  
Вид/Изучение

Требуется информация по той или иной теме. Обычно касается информации о том, как narodmon делает те или иные вещи

  
Вид/Тестирование

Issue or pull request related to testing

  
Вид/Улучшение

Improve existing functionality

  
Приоритет
Критический
The priority is critical

  
Приоритет
Высокий
The priority is high

  
Приоритет
Низкий
The priority is low

  
Приоритет
Средний
The priority is medium

  
Рассмотрено
Дубликат
This issue or pull request already exists

  
Рассмотрено
Не будет исправленно
This issue won't be fixed

  
Рассмотрено
Неверно
Invalid issue

  
Рассмотрено
Подтверждено
Issue has been confirmed

  
Совместимость/Сломанная

Изменение, которое ломает обратную совместимость

  
Статус
Заблокированно
Something is blocking this issue or pull request

  
Статус
Заброшено
Somebody has started to work on this but abandoned work

  
Статус
Требуется больше информации
Feedback is required to reproduce issue or to continue work

  
Тег/NarodMon Совместимость

Всё что связанно с совместимостью IOTишника с NarodMon

  
Хорошая первая issue

Хорошая и не сложная задача для новых контрибьютеров.

No labels
Вид/Баг
Вид/Безопасность
Вид/Документация
Вид/Изучение
Вид/Тестирование
Вид/Улучшение
Приоритет
Критический
Приоритет
Высокий
Приоритет
Низкий
Приоритет
Средний
Рассмотрено
Дубликат
Рассмотрено
Не будет исправленно
Рассмотрено
Неверно
Рассмотрено
Подтверждено
Совместимость/Сломанная
Статус
Заблокированно
Статус
Заброшено
Статус
Требуется больше информации
Тег/NarodMon Совместимость
Хорошая первая issue
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: iotishnik/server#14
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?