forked from signalwire/libks
-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
TEL-6178 Merge 2.0.6 from upstream #4
Open
damirn
wants to merge
235
commits into
development
Choose a base branch
from
jira-tel-6178
base: development
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains 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
* Removed memory pools from ks_json * Renamed cJSON to kJSON to protect against library mismatches * Reworked ks_json interface so it is safer to use
…ixes Strict compilation fixes
…dically can happen when wrapping over (particularly with random sequence starting point for each handle group)
Added check to skip an invalid 0 sequence on new handles, which spora…
…allocated when passed to kws_connect or kws_init then it will use the existing kws object, this allows to assign callbacks or other values before calling connect
…ore calling connect in KWS
Remove ks_handle and ks_throughput
…ent wrapper to the OS thread libs
Remove extra state from ks_thread. It is now just a platform-indepen…
…eads Destroy detached thread on exit
Detached thread must own its own memory pool
[cmake] Don't use CPACK_DEBIAN_PACKAGE_RELEASE
[GHA] Use `cicd-docker-build-and-distribute.yml`.
[GHA] Use `meta-repo-content` workflow.
[GHA] Shorten artifact path store location.
[GHA] Add `arm64v8` build target.
Bump version to 2.0.5
…#214) * kws_raw_write should return the total number of bytes sent, not just the last fragment sent * Adjustment to make sure returns of -1 are captured as well, and fix missing spot in SSL block * Added zero to be inclusive as successful write when using raw sockets, but according to SSL lib 0 is an error so the behavior is slightly different using SSL versus not on the writing returns * Adjust the kws_write_frame so that when it sees a return <=0 it returns the error instead of the total written bytes
[GHA] Remove unsupported build targets.
Bump version to 2.0.6
damirn
requested review from
Elhodred,
dev-ryanc,
baloeng,
minhtuan1407-telnyx and
tajamulTelnyx
January 7, 2025 09:03
dev-ryanc
approved these changes
Jan 7, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, make sure to create tag for v1.10.8-v70 (previous commit)
Elhodred
approved these changes
Jan 7, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.