andreasebner

I think this change would break the symmetrie of the pubsub real time examples (the would not work together anymore). Therefore i would preffer to adjust the help texts in...

Thank you for the contribution @wlkrm! Currently the appveyor build is failing: 5>Compiling... 6>..\src\pubsub\ua_pubsub_networkmessage_json.c(246) : error C[236](https://ci.appveyor.com/project/open62541/open62541/builds/43558754/job/ltrkkjjhc0m7xeri#L236)1: initialization of 'count' is skipped by 'default' label 6> ..\src\pubsub\ua_pubsub_networkmessage_json.c(217) : see declaration...

Please enable your work in at least one unit test.

It's nice to see the progress with the tests. A small hint: It is sufficient to enable the new tests or the flag in one of the existing Github CI-runners...

@jpfr The feature is only availbe with mbedtls and conditional (by an cmake flag). Currently there is a change in a API-function (new paramter). An alternative would be, to add...

This PR looks still valid and ready to merge from an functional point of view. If youre intressted in contributing this feature, it would be nice if you solve the...

I compared your work with the latest master. The change seems to be still valid. How should we proceed?

Hi @jonasgreen88, we looked today again at your PR and we really like to merge your changes. Since this change breaks existing code, we target an integration in the upcoming...

Thank you for updating and rebaseing your PR. The current CI failure is independent of your changes.

@basyskom-jvoe We are currently working on a PR cleanup and we are continuous merging worthy PR's, even if the quite old. It would be great if you can update the...