Add support for Jingle (#22)
In QA
Andrzej Wójcik (Tigase) opened 4 years ago

As we are using slightly adjusted version of Halcyon for project Kangaroo and we do need support for VoIP/WebRTC it would be good to have support for Jingle in Halcyon as it could be then easily used by Kangaroo or future Android or JS client using Halcyon

Andrzej Wójcik (Tigase) commented 4 years ago

@bmalkow If possible, please review my source code which is for now stored in voip branch of Halcyon repository. There is a code to parse SDP by building data blocks, then generating SDP from those blocks. As well there is a code to parse <content/> and other subelements of <jingle/> into data blocks and to serialize those data blocks in to the <content/> element.

If you would be looking at it, please let me know if everything it OK with Kotlin conventions or something should be done differently.

Andrzej Wójcik (Tigase) commented 4 years ago

@bmalkow I've moved all of the code responsible for handling VoIP (Jingle) from TigaseSwift to Halcyon.

There is still code missing related to handling of Jingle session and actual WebRTC integration, but in case of iOS apps, this code is part of the app as it relies on the VoIP implementation and API (WebRTC libraries, iOS notifications/macOS windows, etc).

Should we have a basic (multi account) manager class for handling Jingle sessions in Halcyon or actual reaction on the event should be part of the app? (I would say, that this is an app responsibility to act on the event).

Andrzej Wójcik (Tigase) commented 4 years ago

Implementation is mostly ready but not tested.

issue 1 of 1
Type
Task
Priority
Normal
Assignee
Spent time
22h 30m
Issue Votes (0)
Watchers (0)
Reference
tigase/_libraries/halcyon#22
Please wait...
Page is in error, reload to recover