No bootstrapping when EST feature is disabled about mbed-cloud-client

[teetak01]

There can be some differences between Pelion and Leshan. We'll keep this in mind.

[teetak01]

Hi @ghseb

We disable this feature by default for Mbed OS and the bootstrap flow is the default mode. I do not see how this would be an issue.

https://github.com/PelionIoT/mbed-cloud-client/blob/master/mbed_lib.json#L17-L21

without EST (which is default in most cases) the code goes actually via void M2MInterfaceImpl::bootstrap_finish() (instead of finish_bootstrap()).

[ghseb]

Hello @teetak01

yes, EST disabled by default.

If I remember correctly, the issue in more details was that in

https://github.com/PelionIoT/mbed-cloud-client/blob/master/mbed-client/source/m2minterfaceimpl.cpp#L457-L469

the branch that calls M2MInterfaceImpl::bootstrap_done() is only executed, when the flag M2MInterfaceImpl::_bootstrapped is set to true. Which is not the case. Curiously the _bootstrapped flag is set in M2MInterfaceImpl::bootstrap_done(), but the call to M2MInterfaceImpl::bootstrap_done() depends on _bootstrapped being true.

[teetak01]

Are you doing some inter-operability testing with third-party LwM2M server?

[ghseb]

Well, I used Leshan as Backend, not actually inter-operability testing.

[teetak01]

Should be fixed now with 4.11.0 release.

[ghseb]

Thanks!