Match arguments and property names in UserLoginInfo

[sebastienros]

Without matching names, since there is no default constructor, System.Text.Json deserialization fails and requires a custom converter.

/cc @eiriktsarpalis

[MikeAlhayek]

@sebastienros additionally, why not adding a parameter-less constructor for this class? None of the properties are encapsulated and can be publicly set.

[sebastienros]

Not necessary, and it has required arguments.

[MikeAlhayek]

Not necessary, and it has required arguments.

But helpful. and since all the properties are mutable, I don't see why not.

[sebastienros]

File an issue. Pick your battles.

[eiriktsarpalis]

Not necessary, and it has required arguments.

But helpful. and since all the properties are mutable, I don't see why not.

It would necessitate marking the string properties as nullable, since a default constructor makes null values possible.