2.0.1. types.SampleValue validation Value

[lorenzodonini]

2.0.1. types.SampleValue validation Value

[lorenzodonini]

Hello, thanks for spotting this. It's tricky because in Go the "empty" value for numerical values is literally 0, so passing a 0 value will cause the validation to fail. I understand that passing a zero value is a perfectly valid use case though.

The only thing I can think of is to remove validation for this field altogether. The downside is that an endpoint may send a completely empty json for a sampled value -> it will be considered valid and the parsed value will be 0.0.

[Clint-Mathews]

Hey @lorenzodonini, was checking the documentation though it is a required field the value can be zero.
As you said if removing that validation could cause an empty JSON to go in, can't we add custom validation checking if at least any fields have value?
My suggestion might not the best to do but still, it feels like a possible workaround

Even though all the other fields are optional, if none of the fields including the value is empty, that could be an empty request. Considering a real-time charger scenario, there will be value against a parameter like voltage or power. So a custom validation to check if its empty could be a viable solution.

Referred document: OCPP-2.0.1_part2_specifiction.pdf from Open Charge Alliance.

If the solution looks good I can write up the MR/PR!
Please let me know what you think :)

Also great work on the repo 😇

[lorenzodonini]

Hey @Clint-Mathews, the Value field is not a pointer, so it will always contain a value (fallback being zero). If an empty json actually came in, the parsed value would be 0. Validation always occurs before/after converting to/from json format, which sadly doesn't allow you to determine whether the json was empty.

The best idea that comes to mind is to make the field a pointer type, which could then be correctly validated. That would be a breaking change though.
Or do you have any other concrete ideas for the validation? Totally open to suggestions.