search

openbaton / openbaton-libs

Libraries and sdks shared among projects
Apache License 2.0
3 stars 11 forks source link

Using multiple JSON serialisers is confusing #9

Closed mcilloni closed 7 years ago

mcilloni commented 7 years ago

vnfm-sdk-rest uses Jackson, while the rest of the project uses Gson to serialise/deserialise JSON.

This can lead to an inconsistent and confusing serialisation behaviour, like in VirtualNetworkFunctionRecord, where the endpoint field is marked @JsonIgnore using a Jackson tag, but it's still considered (correctly) when using Gson.

I would consider dropping Jackson to reduce these ambiguities and make the way the project handles JSON more consistent.

marcellom commented 7 years ago

I checked and I agree with the issue. I'll fix it in a specific branch and then I'll refer this issue with the relative pull request.

marcellom commented 7 years ago

This b2a6ccbe4d0ddf64e94c8b09595b2362eb0bbf22 should fix this issue. I will test tomorrow and make a pull request.