fredlcore / BSB-LAN

LAN/WiFi interface for Boiler-System-Bus (BSB) and Local Process Bus (LPB) and Punkt-zu-Punkt Schnittstelle (PPS) with a Siemens® controller used by Elco®, Brötje® and similar heating systems
222 stars 84 forks source link

[BUG] Lockup when querying Parameter 2505 #516

Closed schlaubstar closed 1 year ago

schlaubstar commented 1 year ago

BSB-LAN Version Version: 2.2.1-20221101122308

Architecture ESP32

Bus system BSB

Describe the bug When opening /K34 (which contains the Parameters 2200-2682) the page loads and gets stuck in an endless loop repeating: “2505.0 Kessel - Schwelle Durchflussdetektion: (parameter not supported)” So in the webpage keeps loading and growing in size (vertical scroll), repeating the entry above endlessly … and: locking up the whole device so it needs to be rebooted.

To Reproduce Steps to reproduce the behavior:

  1. Open WebInterface
  2. Click on 'Heizungsfunktionen' (or /K)
  3. Click on 'Kessel' (or /K34)
  4. See error

Logs LAN->HEIZ QUR 2505.0 Kessel - Schwelle Durchflussdetektion: DC C2 00 0B 06 3D 05 1A E1 73 7D HEIZ->LAN ERR 2505.0 Kessel - Schwelle Durchflussdetektion: error 7 DC 80 42 0C 08 05 3D 1A E1 07 9D D5

2505: (parameter not supported)

LAN->HEIZ QUR 2505.0 Kessel - Schwelle Durchflussdetektion: DC C2 00 0B 06 3D 05 1A E1 73 7D HEIZ->LAN ERR 2505.0 Kessel - Schwelle Durchflussdetektion: error 7 DC 80 42 0C 08 05 3D 1A E1 07 9D D5

2505: (parameter not supported)

LAN->HEIZ QUR 2505.0 Kessel - Schwelle Durchflussdetektion: DC C2 00 0B 06 3D 05 1A E1 73 7D HEIZ->LAN ERR 2505.0 Kessel - Schwelle Durchflussdetektion: error 7 DC 80 42 0C 08 05 3D 1A E1 07 9D D5

2505: (parameter not supported)

LAN->HEIZ QUR 2505.0 Kessel - Schwelle Durchflussdetektion: DC C2 00 0B 06 3D 05 1A E1 73 7D HEIZ->LAN ERR 2505.0 Kessel - Schwelle Durchflussdetektion: error 7 DC 80 42 0C 08 05 3D 1A E1 07 9D D5

2505: (parameter not supported)

LAN->HEIZ QUR 2505.0 Kessel - Schwelle Durchflussdetektion: DC C2 00 0B 06 3D 05 1A E1 73 7D HEIZ->LAN ERR 2505.0 Kessel - Schwelle Durchflussdetektion: error 7 DC 80 42 0C 08 05 3D 1A E1 07 9D D5

2505: (parameter not supported)

Expected behavior The page should load displaying parameters 2200-2682 or displaying an erromessage but not locking up in an endless loop.

Screenshots See attached screenshot

Desktop (if applicable, please complete the following information): • Not of relevance: happens independent of the browser/device opening the URL

Smartphone (if applicable, please complete the following information): • Not of relevance: happens independent of the browser/device opening the URL

Additional context Hardware: ESP32 Version: 2.2.1-20221101122308 Freier Speicher: 219232 Bytes Uptime: 185239

The whole device is attached to a Brötje EcoTherm Plus (BBS EVO 20), who could be added as “tested” device.

The Debug-Log (Serial Output) displays various error7 messages before, which do not lead to an lockup but the addition "parameter not supported" in the browser.

BSBLanLockup-Screenshot

fredlcore commented 1 year ago

You are using an outdated version of BSB-LAN, please update to the current version as we don't fix bugs in older versions.

1coderookie commented 1 year ago

@schlaubstar Hmm, didn't you post a comment about an hour ago about where to find the current version..? Anyway - just download it directly from the main repo and please do a complete install from scratch. See also chaps 2 & 3.3 in the manual to note what to do.. https://github.com/fredlcore/BSB-LAN

schlaubstar commented 1 year ago

@1coderookie : Thanks ... Github is missing a PM function. I found out in the manual, that the "latest version" at https://github.com/fredlcore/bsb_lan/archive/master.zip is newer than the "official latest release" found here: https://github.com/fredlcore/BSB-LAN/releases. Maybe you want to update the link. I already did compile the latest version but I try to figure out which BSB_LAN_custom_defs.h parameters I need. The parameter 2505 which caused the issue is by default not visible.

1coderookie commented 1 year ago

Which link should I update? I don't get what you mean - the "master" ist always the current one (so the state of it is like "work in progress"), the "release" is the last 'officially' released version from the master (so the state of that one is like "final").. So right now the latest release is v2.2 where the current master is v3.x.

And what do you mean by this:

I already did compile the latest version but I try to figure out which BSB_LAN_custom_defs.h parameters I need. The parameter 2505 which caused the issue is by default not visible.

If you installed the latest master v3.x, then you need to follow the steps in chapter 3.3 of the manual to get your specific custom_defs.h from @fredlcore - nothing more and nothing less.

schlaubstar commented 1 year ago

@1coderookie : I am sorry for the confusion. My issue was closed with the explanation that I was using an outdated version, that I had downloaded via the official releases link. So I assumed, that the link was pointing to an outdated version, not instantly understanding that you only support bugs on the latest master.

I tried to compile the latest master but it fails for the ESP32 Dev module (which I am using), although I re-verified to have removed the correct directories/renamed the includes and also am on the desired version 2.0.2 of the ESP32 Board definitions as there seems to be an issue with esp32_time.h The compiler complains about a lot of multiple definitions for time objects.

\Arduino\BSB-Lan\BSB-LAN-3.0\BSB_LAN\src\Time/Time.cpp:286: multiple definition ofsetTime(int, int, int, int, int, int)' \Arduino\BSB-Lan\BSB-LAN-3.0\BSB_LAN\src\Time/Time.cpp:133: multiple definition of year()'; \Arduino\BSB-Lan\BSB-LAN-3.0\BSB_LAN\src\Time/Time.cpp:124: multiple definition ofmonth()' ...`

Any ideas, what is wrong here ? Thank you very much for the support.

fredlcore commented 1 year ago

Yes, that's a bug we're fixing right now. Just remove the directory BSB_LAN\src\Time and all should be fine.

schlaubstar commented 1 year ago

Thank you so much. Removing the folder fixed it and also allowed to compile the binaries and do an OTA Update. I am just compiling the text file with available parameters for the custom_defs.h. Thanks again for the support.