Bug in void mocpp_initialize(const char *backendUrl,... about microocpp

[matth-x]

Hi @samuellavoie90, thanks for this report!

Did you already try to change the url parameter of webSocket->begin() accordingly? Did it work?

[samuellavoie90]

Yes, I already did the change: I added a new string called 'Path' under Host_Port_Path, and I concatenated the chargerID to it instead of Url. It worked well, but we only tried the non encrypted socket->begin. Before making the change, I also looked at pretty much the entire code path of socket->begin and it really does expect only the path component of the URL (first '/' included). I suspect the error was introduced when the signature for the mocpp_initialize was changed.

[matth-x]

Thanks for taking a deeper look, will fix it like suggested.

Actually this confused me for years. It is possible to send the full URL in HTTP requests, but it's really uncommon and quite a few WS servers had issues with it. I thought the WS lib had a weird behavior there, but it turns out it was just me 😅

[matth-x]

@samuellavoie90 The fix is ready now: #283

Can you check if it works with your WS server?