fix(modem): CEREG parsing fails due to incorrect state index (IDFGH-16681) (IDFGH-16682)

[JellevanKraaij]

Description

This PR fixes a bug in get_network_registration_state where parsing the AT command response +CEREG: ,,... failed because std::from_chars was attempting to parse the number starting at the preceding comma.

The fix involves incrementing the start position by one (+ 1) before calling std::from_chars to correctly skip the leading comma delimiter.

Related

Fixes #923

Testing

Verified that the function now correctly extracts the integer from typical responses like +CEREG: 2,1

[CLAassistant]

All committers have signed the CLA.

[david-cermak]

LGTM, thanks for the fix 👍

Note that it's been fixed on master, but missed that when publishing v1.4.1

esp-protocols/components/esp_modem/src/esp_modem_command_library.cpp

Lines 651 to 652 in 318bca1

	// Find the end of the state value - either a second comma or end of line
	size_t state_pos_end = out.find(',', state_pos_start + 1);

will merge and create a new patch version.

[JellevanKraaij]

Thanks alot! I dont understand the release branch and how thats managed, but hope you keep track of it!
When will this be available in esp registry @david-cermak ?
Did not see it in main, sorry!

[david-cermak]

When will this be available in esp registry

just released, should be available already

Did not see it in main, sorry!

I am sorry, should have backported before v1.4.1.
I'll create v2.0 soon