Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SYS] ESP32 JSON buffer size fixed for RuuviTag_RAWV2 #1906

Merged
merged 1 commit into from
Mar 4, 2024
Merged

[SYS] ESP32 JSON buffer size fixed for RuuviTag_RAWV2 #1906

merged 1 commit into from
Mar 4, 2024

Conversation

ssalonen
Copy link
Contributor

@ssalonen ssalonen commented Mar 4, 2024 •
edited
Loading

816 bytes is just enough for parsing RuuviTag_RAWV2 json properties.

Resolves #1905

Description:

Checklist:

  • The pull request is done against the latest development branch
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • I accept the DCO.

@ssalonen
ESP32 JSON buffer size fixed for RuuviTag_RAWV2
2204144 
816 bytes is just enough for parsing RuuviTag_RAWV2
json properties.

Resolves 1technophile#1905

Signed-off-by: Sami Salonen <ssalonen@gmail.com>
@ssalonen ssalonen mentioned this pull request Mar 4, 2024
Closed
@1technophile
Copy link
Owner

Thanks

@1technophile 1technophile merged commit 5432016 into 1technophile:development Mar 4, 2024
75 checks passed
@1technophile 1technophile changed the title ESP32 JSON buffer size fixed for RuuviTag_RAWV2 [SYS] ESP32 JSON buffer size fixed for RuuviTag_RAWV2 Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ZgatewayBT::launchBTDiscovery theeng properties json deserialization fails with NoMemory on ESP32
2 participants
@ssalonen @1technophile