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

[Bug] High SSL heap usage can cause reset #153

Open
BlueAndi opened this issue Dec 12, 2023 · 1 comment
Open

[Bug] High SSL heap usage can cause reset #153

BlueAndi opened this issue Dec 12, 2023 · 1 comment
Labels
SW Bug Something isn't working in the software.

Comments

@BlueAndi
Copy link
Owner

A outgoing https:// connection needs about 47 kByte of heap memory. If several plugins requests informations at the same time, it will cause a reset by AsyncTcpSock. AsyncTcpSock uses std::dequeue which allocates memory and causes an system abort.

Idea: Serialize all AsyncHttpClient requests.
@BlueAndi
Copy link
Owner Author

BlueAndi commented Dec 18, 2023 •
edited
Loading

BlueAndi added a commit that referenced this issue Dec 18, 2023
@BlueAndi
Use http:// instead of https:// for less required heap memory for SSL…
0b713d0 
… connection.

#153
BlueAndi added a commit that referenced this issue Dec 18, 2023
@BlueAndi
Memory monitor thresholds updated and considering additional the curr…
be56295 
…ent available heap memory.

#153
BlueAndi added a commit that referenced this issue Dec 18, 2023
@BlueAndi
Use std::nothrow for memory allocation to avoid unhandled exception.
9d8ab2a 
#153
BlueAndi added a commit that referenced this issue Dec 18, 2023
@BlueAndi
Serialize every outgoing connection with AsyncHttpClient to avoid run…
399b584 
…ning out of heap, especially for SSL connections.

#153
BlueAndi added a commit that referenced this issue Dec 19, 2023
@BlueAndi
Paranoia checks added for payload.
eb0d7a0 
#153
@BlueAndi BlueAndi added the SW Bug Something isn't working in the software. label Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
SW Bug Something isn't working in the software.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@BlueAndi