(independent dual-zone) + DuckDNS update…#459
Open
asdafe wants to merge 5 commits into
Open
Conversation
…r + GravityMon entegrasyonu
Author
|
independent control, duckdns integration for remote access and gravitymon battery persentece update |
… yerine) updateIndependentState() hem cooler hem heater kararinda fridgeSensor okumasini kullaniyordu. Isitici tarafi artik beerSensor->readFastFiltered() ile karsilastiriliyor, heaterAllowed kontrolu de beerSensor->isConnected() kullaniyor. Bu bug, Fridge Temp hic beerSetting seviyesine cikmadigi icin isiticinin surekli acik kalmasina sebep oluyordu. Co-authored-by: Cursor <cursoragent@cursor.com>
…cosmetic fix) These 257 files' content is byte-identical to upstream/master; only their file mode had accidentally changed from 100755 to 100644 during a Windows-based commit. No functional change, just cleans up the PR diff. Co-authored-by: Cursor <cursoragent@cursor.com>
Author
|
there is a memory leak i cant find why |
…buffer)) handleBody() mallocs _buffer for every request (even bodyless GETs, since isRequestHandlerTrivial() returns false). Several handleRequest() branches (Tilt/Pill scan commands, SETGRAVITY_PATH, GravityFormulaPath, the GRAVITY_PATH non-POST early return, and the final GET fallback) returned without ever freeing it, permanently leaking the allocation each time (the next handleBody() call overwrites the pointer, so it can never be recovered). Added a single _freeBuffer() helper and call it from every exit path of handleRequest(). Co-authored-by: Cursor <cursoragent@cursor.com>
Keep the early HTTP 200 ACK for hydrometers but stop the second request->send() that orphaned AsyncWebServer responses (~1KB/hour). Also add DiagCounters, richer /fs heap metrics, ws.cleanupClients(), ignore tools/, and document the fixes in README. DuckDNSUpdater.h credentials are intentionally not committed. Co-authored-by: Cursor <cursoragent@cursor.com>
Author
|
fixed the memory leak |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…r + GravityMon entegrasyonu