json-rust about actix-web HOT 9 CLOSED

Why do you want to use streams directly? Actix already uses streams and serde, req.json() just convinience helper with extra checks

https://github.com/actix/actix-web/blob/master/src/json.rs#L50

To fix your compilation bug, just add use futures::Stream to your code

from actix-web.

Hello (and merry Christmas)

with json-rust:

1. I can use several useful functions (is_object, is_array... https://docs.rs/json/0.11.12/json/enum.JsonValue.html#method.is_object) that serde_json does not implement (or poorly documented)
2. transforming into structure is not useful in many cases (therefore less code)
3. faster than serde_json https://github.com/serde-rs/json-benchmark
4. easy for beginners like me
5. why not and for fun

Why do you want everyone to use serde_json? Why don't you implement a req. json_rust () (json-rust) and req. json_serde ().

Ami44

from actix-web.

I see, your reasons are valid. I’ll add example.

req.json() just a convince, I am still playing with api.

from actix-web.

I updated guide and example.

would you provide PR for json-rust crate integration?

from actix-web.

Thank you. I have been discovering rust for 2 weeks and I don't have the level to propose a PR

from actix-web.

but I proposed a PR flosse/rust-web-framework-comparison#36 to improve the visibility of actix-web :-)

from actix-web.

thanks.

btw adding json-rust is easy task, it should be mostly copy from exiting JsonBody implementation

from actix-web.

Could you update your PR and include yes for sessions. Actix has sessions now

from actix-web.

Done

from actix-web.