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

fix offerToReceive(Audio|Video): false #1683

Closed
wants to merge 0 commits into from

Conversation

fippo
Copy link
Contributor

@fippo fippo commented Dec 6, 2017

specifies that offerToReceive* stops all non-stopped transceivers from sending.

@fippo fippo requested a review from fluffy as a code owner December 6, 2017 21:29
@fippo
Copy link
Contributor Author

fippo commented Dec 6, 2017

A stab at #1586

Ironically Firefox Nightly just "broke" this: https://jsfiddle.net/dyytbfxk/
is sendrecv in nightly, sendonly in stable.

Firefox stable sets (to my surprise) the direction on all m-lines.
This shows that this feature is 💩 💩 💩 :-)

@jan-ivar
Copy link
Member

jan-ivar commented Dec 6, 2017

@fippo By "broke" you mean we now implement the spec as mandated?

@fippo
Copy link
Contributor Author

fippo commented Dec 6, 2017

given that the spec decided to change existing behaviour without even bothering to investigate usage (I have not seen httparchive usage data) yes.

It is very much unclear how many people depend on this and the lack of feedback from consumers of the peerconnection API (both service providers and opensource library vendors) is quite concerning.

@fippo
Copy link
Contributor Author

fippo commented Dec 7, 2017

needs rebase after #1686

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants