gloo net add .json convenience method for Request

[databasedav]

Summary

add .json convenience method for Request

Motivation

makes sending json parameters more ergonomic

Detailed Explanation

right now, we must do a .body(serde_json::to_string(&JsonSerializableStruct { ... }).unwrap()), which isn't as ergonomic for an http library as, for example, reqwest with which you can do .json(&JsonSerializableStruct { ... })

Drawbacks, Rationale, and Alternatives

i don't think there are drawbacks, refer to reqwest source for implementation

Unresolved Questions

[databasedav]

you also have to add an .header("Content-Type", "application/json") right now, which would be embedded in the method

[ranile]

Would you like to PR an implementation?

[minauteur]

If something like the following would suffice (not sure given that the surrounding methods return Self rather than Result), I can start adding some tests and open a PR:

    /// convenience method for json request bodies
    pub fn json<T: Serialize + ?Sized>(self, value: &T) -> Result<Self,Error> {
        let json = serde_json::to_string(value)?;
        Result::Ok(
            self.header("Content-Type", "application/json")
            .body(json)
        )
    }

It's been a bit since I've done any Rust--apologies if that is bad 😬

[minauteur]

This can probably be closed, as the requested change was added in #204