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

configure max_content_len per session #4892

Open
Piramma opened this issue Aug 24, 2021 · 0 comments
Open

configure max_content_len per session #4892

Piramma opened this issue Aug 24, 2021 · 0 comments
Labels
enhancement good-first-issue Good for newcomers help-wanted This issue is not being actively worked on, but PRs welcome. size-m Estimated task size: medium (~1w)

Comments

@Piramma
Copy link

Piramma commented Aug 24, 2021

Suggested enhancement

The max_content_len (in and out) is configurable at compile time only. It is used in mbedtls_ssl_setup for alloc memory.

Justification

When in need of having 2 tls sessions to different servers, even if one server is not supporting max_content_len, both sessions are forced to have 16k of IN buffer since it is a compile time option. If configurable at run time, this can save the memory from one session.
@paul-elliott-arm paul-elliott-arm added Community enhancement good-first-issue Good for newcomers help-wanted This issue is not being actively worked on, but PRs welcome. size-m Estimated task size: medium (~1w) labels Aug 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement good-first-issue Good for newcomers help-wanted This issue is not being actively worked on, but PRs welcome. size-m Estimated task size: medium (~1w)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@daverodgman @Piramma @paul-elliott-arm