`health_server_fault_clear` restoring out-of-date publish period

[isundaylee]

Consider the following sequence of events: 

1. `health_server_fault_register` is called, and it saves the current publish period.

2. Config server receives a message and updates publish period of the health server.

3. `health_server_fault_clear` is called, and it restores the previously saved publish period, which is now out-of-date.

Is this the intended behavior? It seems that in step 3 the publish period should be restored to the value set in step 2, instead of the old one that was saved in step 1.

Thanks! 

[tesc]

Hi,

Thank you for reporting this issue Bug reports are always appreciated!

I had a look at the source code and can verify this looks like a  bug. I have notified the nRF5 SDK for Mesh team. Thanks again!

Regards,
Terje

[isundaylee]

Thanks for the update! Looking forward to hearing back from them. 

[tesc]

Hi,

The mesh team has also confirmed this to be a bug now, and they intend to fix it for next release of the nRF5 SDK for Mesh.

Regards,
Terje