Fix: Deserialization errors when using symfonia and creating a user with chorus

[bitfl0wer]

I have encountered the error Deserialization error: Found String \"0\", expected u8 when tinkering with chorus and symfonia. The fix for this is trivial, when using serde_aux, which we already are.

My thought is to apply these deserialize_option_number_from_string and deserialize_number_from_string onto all fields in entity structs, where the field is of a numeric type. What do you think of this, @kozabrada123?

[kozabrada123]

I see no real problems with this, however, wouldn't it be better to fix the serialization in symfonia?

[kozabrada123]

or are all postgres uints always serialized as strings, in case they are too large?

[bitfl0wer]

or are all postgres uints always serialized as strings, in case they are too large?

Postgres does not have sized unsigned integers - hence my crate sqlx_pg_uint.

I see no real problems with this, however, wouldn't it be better to fix the serialization in symfonia?

What is weird is, that symfonia just uses the User type from chorus, then wraps that in the poem Json type, and sends that over the wire with, as far as I can tell, no additional modifications. Needs further investigation I think

[kozabrada123]

Postgres does not have sized unsigned integers - hence my crate sqlx_pg_uint.

Uhm yes, I meant, are those types from your crate meant to be serialized as strings? xD

What is weird is, that symfonia just uses the User type from chorus, then wraps that in the poem Json type, and sends that over the wire with, as far as I can tell, no additional modifications. Needs further investigation I think

We should check, does just using serde_json also produce strings for those types?

[bitfl0wer]

Now that you mentioned it, sqlx_pg_uint uses the De-/Serialize implementations of the underlying BigInt type. Those are very likely Strings and thus the cause of the problem. I'll update the crate and override the De-Serializations by using proper uint types there as well

[bitfl0wer]

Fixed by updating sqlx_pg_uint 👍