track id uuid postfix prevents association in amazon-chime-sdk-js

[ian-luca]

YOU MUST read first!

Please use Community Forum for general technical discussions and questions.

• I have used Google with the error message or bug in association with the library and Cordova words to make sure the issue I'm reporting is only related to iOSRTC.
• I have provided steps to reproduce (e.g. using sample app code https://github.com/cordova-rtc/cordova-plugin-iosrtc-sample or updated extra/renderer-and-libwebrtc-tests.js file).
• I have provided versions of third party library name, ios, Xcode and plugin version and adapter.js version if used.

Note: If the checkboxes above are not checked (which you do after the issue is posted), the issue will be closed, removing this checkbox will result in automatic closed issue.

Versions affected

• Cordova version: 10.0.0
• Cordova iOS version: 6.2.0
• Plugin version: 6.0.18
• iOS version: 14.4
• Xcode version: 12.4 (12D4e)
• WebRTC-adapter version: -
• WebRTC Framework version: amazon-chime-sdk-js 2.4.0

Description

When using cordova-plugin-iosrtc with amazon-chime-sdk-js the iOS user cannot see other meeting participants. It seems that in the plugin track ids are appended by uuids which makes it impossible for the chime sdk to associate the there called subscription acknowledgement tracks with the actual tracks. (maybe related to #432?)

Steps to reproduce

In short: join a meeting with other participants

When joining a chime meeting, chime subscribes to other users tracks (https://github.com/aws/amazon-chime-sdk-js/blob/a8a5413688e04e57266fc9c4da709b6a89bb7a35/src/task/SubscribeAndReceiveSubscribeAckTask.ts#L92) where tracks are referenced as track_video or track_video_13 for example. At this point in https://github.com/aws/amazon-chime-sdk-js/blob/a8a5413688e04e57266fc9c4da709b6a89bb7a35/src/videostreamindex/DefaultVideoStreamIndex.ts#L85 they are associated with the meeting attendee. So far so good.
But at the "received track event" track ids are postfixed with a uuid and can therefore not be associated with an attendee or a stream.

Expected results

track ids should not be postfixed with a uuid. Or atleast there should be an option to configure it in some way to work with multiple sdks/libs.

Actual results

track ids cannot be associated to what the lib originally intended.

[hthetiot]

Can you comment the line that check the length is above 36, or replace by if (0) see if removing "bad" UUID prefix some your issue before we made change on iosrtc.

https://github.com/cordova-rtc/cordova-plugin-iosrtc/blob/master/src/PluginMediaStream.swift#L24

Look for the same in PluginMediaStreamTrack.

If you confirm this fix you issue, i may patch or offer more control over this behavior.

[ian-luca]

Thanks for the quick reply! Just tested by setting the check < 0 and can confirm it now works as expected.

[hthetiot]

@ian-luca can you provide example values of stream and track id for aws-shim so we can improve the bad IDs detection.

[ian-luca]

@hthetiot I don't know much about the naming logic but from observation the first video track seems do get the id track_video, all other tracks will be named something like track_25 where 25 is the streamId

stream ids are simple numers as in the example above

[hthetiot]

This is really bad, trackId and streamId are supposed to be GUID.

• https://tools.ietf.org/id/draft-ietf-mmusic-msid-17.html

[hthetiot]

Here is the possible fix @ian-luca see test instruction in PR description:

• implement PREFFIX_INVALID_MEDIA_ID pluxin.xml preference to disable UUID preffix for bad media ids via config #647 #648

If you can test that would help a lot, if it works i will release in 6.0.19