fix: redis, /get & /post parsing #15

DarkCat09 · 2024-06-03T10:29:12+03:00

DarkCat09 commented

2024-06-03 10:29:12 +03:00

No description provided.

DarkCat09 force-pushed dev/dc09 from e95d570709 to 1c0437b820

2024-06-03 11:01:53 +03:00

Compare

DarkCat09 changed title from ~~WIP: dev/dc09~~ to dev/dc09

2024-06-03 11:02:31 +03:00

DarkCat09 changed title from ~~dev/dc09~~ to fix: redis, /get & /post parsing

2024-06-03 11:03:34 +03:00

DarkCat09 requested review from nm17 2024-06-03 11:03:54 +03:00

nm17 requested changes 2024-06-03 14:16:17 +03:00

Dismissed

nm17 left a comment

Подправь это и приму без проблем

src/web_server/old_device_sensor_api/mod.rs Outdated

					
				@ -51,3 +51,3 @@

				            },

				            "application/x-www-form-urlencoded" => {

				                match serde_qs::from_bytes::<NMDeviceDataPacket>(body.as_ref()) {

				                match qs_parser::parse_nm_qs_format(&body.as_ref().to_str_lossy()).await {

nm17 commented

2024-06-03 13:58:03 +03:00

Может быть всё-таки std::str::from_utf8 используешь? Я не хочу работать с не utf-8 данными, так что заменять невалидные вещи на � не лучшее решение.

Может быть всё-таки `std::str::from_utf8` используешь? Я не хочу работать с не utf-8 данными, так что заменять невалидные вещи на � не лучшее решение.

DarkCat09 marked this conversation as resolved

DarkCat09 added 1 commit 2024-06-03 15:14:03 +03:00

style: error on invalid utf8 seq 803acce7e9

DarkCat09 requested review from nm17 2024-06-03 15:14:22 +03:00

DarkCat09 added 1 commit 2024-06-03 15:37:37 +03:00

style: cleanup UTF8 error throwing code 19a4e0c299

nm17 approved these changes 2024-06-03 16:22:41 +03:00

nm17 merged commit 87e189580e into master

2024-06-03 16:23:02 +03:00

nm17 referenced this pull request from a commit

2024-06-03 16:23:04 +03:00

Merge pull request 'fix: redis, /get & /post parsing' (#15) from dev/dc09 into master