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

Receive Transport Crash on second call #20

Open
lets-swapcode opened this issue Sep 11, 2023 · 3 comments
Open

Receive Transport Crash on second call #20

lets-swapcode opened this issue Sep 11, 2023 · 3 comments

Comments

@lets-swapcode
Copy link

lets-swapcode commented Sep 11, 2023

PS - Tried almost everything. Checked discourse group as well.

Case - First time everything works fine, able to leave the call as well BUT when trying to join again then its crashes in the beginning with below error

Screenshot 2023-09-11 at 2 47 28 PM

On leave -

  1. Attempt 1 - Tried setting to nil to all the property (consumer, producer, track, media, device close and nil) and socket disconnect
  2. Attempt 2 - Only socket disconnect
  3. Attempt 3 - consumer close, sent pause consumer socket and socket disconnect

What is the correct way to do this?

Transport callback handling
`
func onConnectionStateChange(transport: Transport, connectionState: TransportConnectionState) {
LOG.debug("Transport- Reconnect- (transport.id == sendTransport?.id ? "send" : "receive") onConnectionStateChange == (connectionState)")

    qJMMediaBGQueue.async {
        if connectionState == .disconnected && self.isCallEnded{
            LOG.error("Transport- End- transport closed")
            transport.close()
            return
        }
        
        if connectionState == .closed && transport.id == self.sendTransport?.id{
            LOG.error("Transport- End- transport sent closed - stop capture")
            self.videoCapture?.stopCapture()
        }
    }

`

@lets-swapcode
Copy link
Author

woah!! With this changes #16 , this crash got resolved too.

Thanks man!!

One last time please add support for iOS 12. Just need to update the minimum deployment, nothing else. Thank you.
#18

@lets-swapcode
Copy link
Author

Nope, Today it started crash again on transport close.

Is m112 version of WEBRTC integrated with mediasoup is stable or should we use m94?

@lets-swapcode lets-swapcode reopened this Sep 14, 2023
@lets-swapcode
Copy link
Author

I had to remove the calling of close to remove the crash.

Please let us know what is the proper way to handle this?

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

No branches or pull requests

1 participant