-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
Fix #169 : missing pids in RTCP and DESCRIBE messages when transforming #179
base: master
Are you sure you want to change the base?
Conversation
… transforming When doing the translation the pid list is empty. This fixes this issue.
Hi @Barracuda09 , This fixes this: #169 (comment) |
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.
Thanks for looking into this!
I have to look at it some more to see why this is needed, as I thought it was already fixe
Hi @Barracuda09 , The problem (I feel) is because the current implementation of the "virtual" frontend used for the transformation is not "updated". See this part of the code (as example): In the concrete DeviceData Regards. |
Hi @lars18th I have looked and tested this without this fix, but here it seems to work as fixed with 7107fa5 I have a test with this translation:
With this request:
I get the PIDs in the describe
Here you can see that the original request with freq and pids are filled in |
When doing the translation the pid list is empty. This fixes this issue.