search

CurtRod / SimpleEVSE-WiFi

Brings EVSE WB/DIN into your WiFi
https://www.evse-wifi.de
78 stars 31 forks source link

small fix: log entries are evicted by FIFO, not LIFO #75

Closed J4nsen closed 2 months ago

J4nsen commented 2 years ago

LIFO is a stack. here, log entires are removed in FIFO manner.

edsub commented 9 months ago

@J4nsen Did you manage to compile your fork (incl Log fix) into a deployable firmware file? I do not master programming n C and I am suffering from a bug that crashes/reboots the charger when the Log is opened/requested (currently on EVSE-WiFi 1.1.1 as available on https://www.smartwb.de/firmware-downloads/). So getting a working firmware would be great. Can you help me get that?

J4nsen commented 9 months ago

Hi edsub,

my change was purely cosmetic, so I didnt try to compile the firmware. :/ Sorry, cant help you there

edsub commented 9 months ago

Thx for response. Are you experiencing the software to crash when you access the log (via web gui or api)? Because that is my issue. And frankly I cannot believe this has not been fixed if the issue is with all users (meaning: It can be just my setup / ESP setup causing this).

J4nsen commented 9 months ago

I can access the log without crashing the system. Perhaps you have a broken log entry which triggers the crash.