Skip to content

Conversation

wangzhen263
Copy link

When I add h264 in our repo via sdpFilter, it never runs. Then I found it is overwritten by the sdpfilter.

@nathanoehlman
Copy link
Contributor

Hey Jeff - yep, can definitely see how that could be problematic.

I'm wondering if instead of running them both (which might also cause confusion) we just throw an error if both are configured and then put the responsibility for determining which option to pass back onto the implementer.

Alternately, if you think there is value in being able to chain sdpfilters, it might be better to handle things by reducing over an array.

Thoughts?

@silviapfeiffer
Copy link
Member

I'd suggest reducing over the array. Just my 2c worth.

@silviapfeiffer
Copy link
Member

@nathanoehlman maybe you could update the PR with what you think?

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.

3 participants