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

Consistent panic on publisher join #450

Closed
tab1293 opened this issue Mar 1, 2021 · 3 comments
Closed

Consistent panic on publisher join #450

tab1293 opened this issue Mar 1, 2021 · 3 comments

Comments

@tab1293
Copy link
Contributor

tab1293 commented Mar 1, 2021

Your environment.

  • Version: v1.9.1
  • Client: v1.6.0
  • Environment: K8S
  • Are you using a TURN server? Coturn
  • Other Information
[2021-03-01 22:37:09.657] [DEBUG] [128][peer.go][func3] => on publisher ice candidate called for peer 960f7958-25bc-47f6-b0c7-85eb30c85a50
[2021-03-01 22:37:09.692] [DEBUG] [128][peer.go][func3] => on publisher ice candidate called for peer 960f7958-25bc-47f6-b0c7-85eb30c85a50
[2021-03-01 22:37:09.692] [DEBUG] [128][peer.go][func3] => on publisher ice candidate called for peer 960f7958-25bc-47f6-b0c7-85eb30c85a50
[2021-03-01 22:37:09.693] [DEBUG] [128][peer.go][func3] => on publisher ice candidate called for peer 960f7958-25bc-47f6-b0c7-85eb30c85a50
[2021-03-01 22:37:09.695] [DEBUG] [128][peer.go][func3] => on publisher ice candidate called for peer 960f7958-25bc-47f6-b0c7-85eb30c85a50
[2021-03-01 22:37:09.894] [DEBUG] [63][publisher.go][func3] => ice connection state: connected
[2021-03-01 22:37:09.902] [DEBUG] [101][peer.go][func1] => peer 960f7958-25bc-47f6-b0c7-85eb30c85a50 negotiation needed
[2021-03-01 22:37:09.904] [INFO] [110][peer.go][func1] => peer 960f7958-25bc-47f6-b0c7-85eb30c85a50 send offer
[2021-03-01 22:37:09.905] [DEBUG] [116][peer.go][func2] => on subscriber ice candidate called for peer 960f7958-25bc-47f6-b0c7-85eb30c85a50
[2021-03-01 22:37:09.915] [DEBUG] [116][peer.go][func2] => on subscriber ice candidate called for peer 960f7958-25bc-47f6-b0c7-85eb30c85a50
[2021-03-01 22:37:09.916] [DEBUG] [116][peer.go][func2] => on subscriber ice candidate called for peer 960f7958-25bc-47f6-b0c7-85eb30c85a50
[2021-03-01 22:37:09.917] [DEBUG] [116][peer.go][func2] => on subscriber ice candidate called for peer 960f7958-25bc-47f6-b0c7-85eb30c85a50
[2021-03-01 22:37:09.918] [DEBUG] [116][peer.go][func2] => on subscriber ice candidate called for peer 960f7958-25bc-47f6-b0c7-85eb30c85a50
[2021-03-01 22:37:10.002] [DEBUG] [57][subscriber.go][func1] => ice connection state: checking
[2021-03-01 22:37:10.155] [DEBUG] [48][publisher.go][func1] => Peer 960f7958-25bc-47f6-b0c7-85eb30c85a50 got remote track id: 2c559dd0-cfbb-4ebd-a255-88e46b1c3448 mediaSSRC: 2736473610 rid : streamID: f98e5a49-3179-45d2-89f9-0283ad829132
[2021-03-01 22:37:10.155] [DEBUG] [180][router.go][AddReceiver] => Setting feedback goog-remb
[2021-03-01 22:37:10.155] [DEBUG] [180][router.go][AddReceiver] => Setting feedback transport-cc
[2021-03-01 22:37:10.156] [DEBUG] [180][router.go][AddReceiver] => Setting feedback nack
[2021-03-01 22:37:10.156] [DEBUG] [180][router.go][AddReceiver] => Setting feedback nack
panic: runtime error: index out of range [1] with length 0

goroutine 243 [running]:
encoding/binary.bigEndian.PutUint16(...)
  /usr/local/go/src/encoding/binary/binary.go:106
github.com/pion/ion-sfu/pkg/buffer.(*Bucket).push(...)
  /go/pkg/mod/github.com/pion/ion-sfu@v1.9.1/pkg/buffer/bucket.go:59
github.com/pion/ion-sfu/pkg/buffer.(*Bucket).AddPacket(0xc000595c80, 0xc0005fb100, 0xf4, 0xf4, 0xc000016f73, 0x0, 0x10e7852, 0x17, 0x0, 0x0)
  /go/pkg/mod/github.com/pion/ion-sfu@v1.9.1/pkg/buffer/bucket.go:37 +0x5b2
github.com/pion/ion-sfu/pkg/buffer.(*Buffer).calc(0xc00045ed80, 0xc0005fb100, 0xf4, 0xf4, 0x166858ea65114ef7)
  /go/pkg/mod/github.com/pion/ion-sfu@v1.9.1/pkg/buffer/buffer.go:304 +0x4e7
github.com/pion/ion-sfu/pkg/buffer.(*Buffer).Bind(0xc00045ed80, 0xc00058f2c0, 0x3, 0x4, 0xc00051a1e0, 0x1, 0x1, 0x1f40)
  /go/pkg/mod/github.com/pion/ion-sfu@v1.9.1/pkg/buffer/buffer.go:169 +0x5a5
github.com/pion/ion-sfu/pkg/sfu.(*router).AddReceiver(0xc000138640, 0xc00051a360, 0xc0000b2c60, 0x0, 0x0, 0x0)
  /go/pkg/mod/github.com/pion/ion-sfu@v1.9.1/pkg/sfu/router.go:180 +0x7d5
github.com/pion/ion-sfu/pkg/sfu.NewPublisher.func1(0xc0000b2c60, 0xc00051a360)
  /go/pkg/mod/github.com/pion/ion-sfu@v1.9.1/pkg/sfu/publisher.go:49 +0x2a5
created by github.com/pion/webrtc/v3.(*PeerConnection).onTrack
  /go/pkg/mod/github.com/pion/webrtc/v3@v3.0.10/peerconnection.go:451 +0x165

What did you do?

Publisher joined a session

What did you expect?

Publisher's tracks to be forwarded correctly

What happened?

ion-sfu panics consistently with the error above

@OrlandoCo
Copy link
Collaborator

OrlandoCo commented Mar 1, 2021

@tab1293 There was an update in the config file, did you update it on your side?, make sure you got this value set right:

# max number of video tracks packets the SFU will keep track
maxpackettrack = 500

@tab1293
Copy link
Contributor Author

tab1293 commented Mar 2, 2021

That fixed the issue, I was still using maxbuffertime. Maybe there could be a default value here (with a log warning) as to not create this issue for others coming from an old config?

@tab1293 tab1293 closed this as completed Mar 2, 2021
@OrlandoCo
Copy link
Collaborator

@tab1293 you are right going to fix it!

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

2 participants