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

feat(chart): add jigasi #99

Merged
merged 4 commits into from
Jan 23, 2024
Merged

feat(chart): add jigasi #99

merged 4 commits into from
Jan 23, 2024

Conversation

emrahcom
Copy link
Member

This PR adds jigasi.

jigasi allows SIP clients to join Jitsi meetings and transciption. No need to publicly open ports if SIP server allows media traffics over itself.

Even some use-cases are tricky as mentioned in this issue, there are some other uses-cases which work with simple settings.

Copy link
Collaborator

@spijet spijet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @emrahcom! Thank you for your PR 🧡

I have a couple of questions/nitpicks, if you don't mind. Please check the review. :)

@spijet
Copy link
Collaborator

spijet commented Jan 22, 2024

Do I understand it right, that Jigasi in this case doesn't need to receive any incoming connections from the outside world? How exactly is it going to communicate with the SIP server and receive incoming calls?

@emrahcom
Copy link
Member Author

Do I understand it right, that Jigasi in this case doesn't need to receive any incoming connections from the outside world? How exactly is it going to communicate with the SIP server and receive incoming calls?

No, it can still receive incoming calls because it is registered SIP server and since SIP server is publicly accessible, jigasi is also publicly accessible through SIP server.

By the way, I tested this use-case in my deployment and it works. I created a jigasi instance by using this Helm Charts and I have no open ports for this instance but the incoming call and media traffic through SIP server work.

@spijet
Copy link
Collaborator

spijet commented Jan 23, 2024

By the way, I tested this use-case in my deployment and it works. I created a jigasi instance by using this Helm Charts and I have no open ports for this instance but the incoming call and media traffic through SIP server work.

Hmm, that's interesting. I was weary of trying Jigasi out myself since I don't have any SIP infrastructure around, and the official Jitsi Meet on Docker documentation mentions something about ports 20000~20050/UDP needed for it.

@spijet
Copy link
Collaborator

spijet commented Jan 23, 2024

Anyway, LGTM. Let's merge it and wait for @at-platform24 to finalize #92 before tagging a new release.

@spijet spijet merged commit 867df9f into jitsi-contrib:main Jan 23, 2024
@emrahcom emrahcom deleted the jigasi branch January 24, 2024 06:12
@emrahcom
Copy link
Member Author

By the way, I tested this use-case in my deployment and it works. I created a jigasi instance by using this Helm Charts and I have no open ports for this instance but the incoming call and media traffic through SIP server work.

Hmm, that's interesting. I was weary of trying Jigasi out myself since I don't have any SIP infrastructure around, and the official Jitsi Meet on Docker documentation mentions something about ports 20000~20050/UDP needed for it.

If the SIP server doesn't allow media traffic (RTP) over itself then these port should be publicly open on Jigasi but this is not the only option.

And jigasi is also used for other purpose such as transcription.

@Yerachmiel-Wand
Copy link

Hi, about

And jigasi is also used for other purpose such as transcription.

Would you mind pointing to any docs explaining how to setup jigasi for transcription with the helm chart?
I don't need SIP, only the ability to generate transcriptions from calls, even not real time.

Thank you. :)

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

Successfully merging this pull request may close these issues.

3 participants