[ESP32] ConnectivityManagerImpl — should UpdateInternetConnectivityState() happen if PostEvent() fails?

[kpschoedel]

Problem

In src/platform/ESP32/ConnectivityManagerImpl.cpp, there are several functions that both call PlatformMgr().PostEvent() and UpdateInternetConnectivityState() (in either order). With PR #9573, PostEvent() returns an error if it fails. It's not obvious whether UpdateInternetConnectivityState() should be called in such cases.

[yufengwangca]

@gjc13 could you help to take a look?

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

[dhrishi]

We will check this cc @shubhamdp