Derive Clone, Serialize and Deserialize for GraphQLRequest and GraphQLResponse

[thomas-jeepe]

For me I need to persist HTTP GraphQL requests and response which is why I need Serialize and Deserialize and then I need to pass them around cloning them.

In order to do so for GraphQLRequest every field is already Serializeable and a simple #[derive(Serialize)], it derives the other traits.

For GraphQLResponse in order to do Deserialize, a custom Deserialize implementation would have to be done. On top of that Clone seems impossible with the GraphQLResponse having borrowed content in the form of a lexer Token<'a> from a parse error.

Changing Token to owning the Name variant would make it possible to clone the response around.

Thats all. It should be relatively simple to implement.

[thomas-jeepe]

So, I was in the process of removing the borrowed content in Token::Name but it seems like the ast, lexer and parser are all dependent on that lifetime and the borrowed content. There might be another approach to it, however for now in order to clone, I will serialize then clone the string.

I do comparisions of GraphQLResponses so a serialized String would likely be used anyways.

I made a pr for the GraphQLRequest.

[tyranron]

In order to do so for GraphQLRequest every field is already Serializeable and a simple #[derive(Serialize)], it derives the other traits.

This was done in 6d345b6 (yeah, buy you 🙃).

So, I was in the process of removing the borrowed content in Token::Name but it seems like the ast, lexer and parser are all dependent on that lifetime and the borrowed content. There might be another approach to it, however for now in order to clone, I will serialize then clone the string.

Actually, the lifetime was stripped from GraphQLResponse in #1081 already.

For GraphQLResponse in order to do Deserialize, a custom Deserialize implementation would have to be done. On top of that Clone seems impossible with the GraphQLResponse having borrowed content in the form of a lexer Token<'a> from a parse error.

I'm not sure how much Deserialize has sense for GraphQLResponse, however, Clone and PartialEq seems to be something possible now, after #1081. Let's try it.