Pre-Submission Checklist

Commit message has the format required by CONTRIBUTING guide
Commits are split per component (core, individual modules, libs, utils, ...)
Each component has a single commit (if not, squash them into one commit)
No commits to README files for modules (changes must be done to docbook files
in doc/ subfolder, the README file is autogenerated)

Type Of Change

Small bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds new functionality)
Breaking change (fix or feature that would change existing functionality)

Checklist:

PR should be backported to stable branches
Tested changes locally
Related to issue #3823

Description

This PR aims to implement what was discussed in mailing list regarding some tls.reload and increasing memory usage.

It adds a new parameter enable_shared_ctx in tls module that if set to 0, preserves the old behavior and if set to 1 (other than 0 tbh), it creates a single SSL context that is being shared. This have the effect of using way less memory when initialized as well, but also minimizes (can't say it fixes the problem) the tls.reload memory increase.

I have also added a small markdown (comparison.md) file, where some comparisons where made between enabled/disabled shared context and with/without CA file (where the initial problem was occurring by the reporter).

Feedback would be necessary to verify whether this patch, acts as expected and kamailio works as intented.

You can view, comment on, or merge this pull request online at:

https://github.com/kamailio/kamailio/pull/3972

Commit Summary

e53364e tls: Add parameter for shared contexts
8ee12d1 tls: Comparison for enable_shared_ctx

File Changes

(4 files)

Patch Links:

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.