Skip to content

Commit df3b8e0

Browse files
authored
Merge pull request #3705 from nextcloud/backport/3670/stable-18.1
[stable-18.1] add compatibility for janus 1.x
2 parents ccda7f2 + 9cf0388 commit df3b8e0

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

app/src/main/java/com/nextcloud/talk/webrtc/PeerConnectionWrapper.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -398,8 +398,7 @@ public void onBufferedAmountChange(long l) {
398398
@Override
399399
public void onStateChange() {
400400
if (dataChannel != null &&
401-
dataChannel.state() == DataChannel.State.OPEN &&
402-
"status".equals(dataChannel.label())) {
401+
dataChannel.state() == DataChannel.State.OPEN) {
403402
sendInitialMediaStatus();
404403
}
405404
}
@@ -535,10 +534,12 @@ public void onRemoveStream(MediaStream mediaStream) {
535534

536535
@Override
537536
public void onDataChannel(DataChannel dataChannel) {
538-
if ("status".equals(dataChannel.label()) || "JanusDataChannel".equals(dataChannel.label())) {
539-
PeerConnectionWrapper.this.dataChannel = dataChannel;
540-
PeerConnectionWrapper.this.dataChannel.registerObserver(new MagicDataChannelObserver());
537+
if (PeerConnectionWrapper.this.dataChannel != null) {
538+
Log.w(TAG, "Data channel with label " + PeerConnectionWrapper.this.dataChannel.label()
539+
+ " exists, but received onDataChannel event for DataChannel with label " + dataChannel.label());
541540
}
541+
PeerConnectionWrapper.this.dataChannel = dataChannel;
542+
PeerConnectionWrapper.this.dataChannel.registerObserver(new MagicDataChannelObserver());
542543
}
543544

544545
@Override

0 commit comments

Comments
 (0)