Zaid-Ajaj/Fable.Remoting

TimeSpan Greater than 2,147,483,647 ms throws error

Closed this issue · 1 comments

The parsing of the TimeSpan type is transferring the value in terms of number of milliseconds stored as int. This will cause an error when trying to make an RPC call with a TimeSpan who's number of milliseconds exceed the range of int.

I believe the issue is here:

let json = serializer.Deserialize(reader, typeof<int>) :?> int

I believe that it can be fixed by converting to a float instead of an int. I believe this per the docs here:
https://docs.microsoft.com/en-us/dotnet/api/system.timespan.frommilliseconds?view=net-5.0

I will see if I can create my own PR to address the issue. I'll reference this issue in the PR.

Fixed in #277 thanks again 🙏 I'll add some integration tests later for this