-
Notifications
You must be signed in to change notification settings - Fork 21
get ice-lite returning nil #217
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
Conversation
Hi @OctopusRage! Thanks for the catch! I think that we could even get rid of the first getter i.e. always look for |
And see if it works for you? |
sure, updated |
Thanks! Btw. can I ask what's your use-case for ice-lite? |
Whatsapp call uses ice lite on their side |
But even without this fix, everything should work, I believe? The worst case scenario is that there will be ICE role conflict that should be automatically resolved |
Yea actually it still doesnt fix problems on my server . |
Can you share this TLS error? How do you observe it? Also is Whatsapp WebRTC API somewhere available? |
TLS :client: In state :connection received SERVER ALERT: Fatal - Close Notify Im sorry whatsap rtc docs still not available for public yet |
close notify is an alert sent by a side that is closing the connection in a graceful way. When you close a tab in a web browser it should also send DTLS close notify. So this error might not tell anything specific. See here: https://datatracker.ietf.org/doc/html/rfc5246#section-7.2.1 Can you confirm that you are receiving any packets? Also, are you sure you are sending any packets? Some services may close the connection when they don't receive any traffic. |
Youre right, they immediately close connection after 20 sec if thereis no packet sent Do i probably need to setup turn server on my local as well? |
I don't think so. If you are able to establish the connection, TURN shouldn't be needed. I would make sure you don't duplicate any actions. E.g. you don't send two the same HTTP requests to their API gateway as this may also be treated as an error |
i dont think so, all http request done before call estabilished
|
Hmm, not sure why there is nil but I actually wonder what this close notify come from. This does not look like our library |
i guess ill try to ask meta team too |
seems like your guess was right |
You can check against |
fix get ice lite not recognized sdp offer that has "a=ice-lite"