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
1.05.04 json encoder fixes #2376
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2376 +/- ##
===========================================
+ Coverage 36.66% 53.82% +17.16%
===========================================
Files 319 334 +15
Lines 63015 64559 +1544
Branches 12952 13264 +312
===========================================
+ Hits 23102 34747 +11645
+ Misses 37115 26058 -11057
- Partials 2798 3754 +956 ☔ View full report in Codecov by Sentry. |
Proposed changes
The JSON decoder now checks for JSON string in addition to handling a JSON object. If a JSON string it assumes the NodeId/ExpandedNodeId/QualfiiedName is formatted as specified in Part 6.
The JSON encoder has a flag to force the serialization of Strings instead of JSON Objects.
The JSON decode now allows an incoming message to have a different namespace table that the server.
Types of changes
Enhancement (non-breaking change which adds functionality)