-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deserializing u128
errors when inside an untagged enum
#46
Comments
At a glance, this doesn't seem easily solvable. @hashedone any ideas? |
The error we are getting with the above test is
which is generated by serde: https://github.com/serde-rs/serde/blob/v1.0.147/serde_derive/src/de.rs#L1667-L1677. But I don't understand what those The raw JSON is
which looks solid. |
@webmaster128 In my investigation, I figured that when deserializing into an The problem is there's no logic for |
I looked a bit deeper into |
I wonder if #54 solves this issue |
This is not fixed by #54. While digging into it, I found these issues (and a few others that probably have the same root cause): |
Minimal reproducible example below. Note that the second you change
u128 -> u64
, the test passes.The text was updated successfully, but these errors were encountered: