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.

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

Add modifiers for RTP TTL in the U/L commands #59

Open
wants to merge 28 commits into
base: master
Choose a base branch
from

Conversation

razvancrainea
Copy link
Contributor

This allows the Media Signalling server to tune the RTP TTL in for each stream.

razvancrainea and others added 28 commits February 13, 2017 10:35
Initially there were 5 statistics, but the number changed after these
commits:
4b099c8 - introduced 4 new stats
9c843e7 - introduced 1 new stat
of the travis build stucking trying to upgrade tcpdump package.
This allows the Media Signalling server to tune the RTP TTL in for each
stream.
of the travis build stucking trying to upgrade tcpdump package.
@razvancrainea
Copy link
Contributor Author

Sorry for not rebasing properly. It it's an issue, I can re-create the pull request on a clean fork.

@razvancrainea
Copy link
Contributor Author

Hey, @sobomax - I want to revive this PR - should I make a new PR against latest master?

@sobomax
Copy link
Member

sobomax commented May 11, 2021

Hi @razvancrainea , thanks for raising it! Fell off my radar sort of. :( I would look into converting it into sub-command for the U/L i.e. U/Lfoobar && Txyz and adding a functional test to make it complete. I can look into the former if you take care of the latter. Thanks!

@razvancrainea
Copy link
Contributor Author

OK, let's do this - let me know when you get a chance to modify the command, and I will soon come back with some tests for it. Cheers!

sobomax added a commit that referenced this pull request May 24, 2021
in the TTL setting sub-command and other places.

PR:	#59
sobomax added a commit that referenced this pull request May 24, 2021
sobomax added a commit that referenced this pull request May 24, 2021
sobomax added a commit that referenced this pull request May 24, 2021
@sobomax sobomax force-pushed the master branch 6 times, most recently from 55700ad to 2131bec Compare January 13, 2023 15:11
sobomax added a commit that referenced this pull request Jan 23, 2023
sobomax added a commit that referenced this pull request Jan 23, 2023
sobomax added a commit that referenced this pull request Feb 2, 2023
sobomax added a commit that referenced this pull request Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants