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

Update of the softphone librairy to work with asterisk (Digest Auth updated) #76

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

pascal-pro
Copy link
Contributor

Update the struct of Softphone and update the Digest Auth part to simplify use.
More info here: https://en.wikipedia.org/wiki/Digest_access_authentication

Pascal Benoit and others added 5 commits December 17, 2020 19:41
Example demonstrates how to connect to FreeSWITCH and save to disk

Co-authored-by: Sean DuBois <sean@siobud.com>
Fix assert-contibutors
@Sean-Der
Copy link
Member

Hey @pascal-ace

Do you mind rebasing? Since you are behind master we have a bunch of conflicts.

How is the project going with these changes! Everything working, any bugs?

@pascal-pro
Copy link
Contributor Author

I will try to make a separate branch, but normally, the conflicts are only in the SIP example, everything else is to refuse. (The hand has also changed its name)
This is a current update, to make it work with asterisk. For the moment, it still works with Freeswitch, but still not with asterisk.
The softphone folder has been updated to add / change / clean up the library and make it more readable and clearer.

@pascal-pro
Copy link
Contributor Author

Maybe you prefer that I host the softphone on my github, and that I make a version, so that the example always works, and I make you PR every time I change the lib or the structures?

@saiteja-v
Copy link

These changes seems to have an issue in register.go
Code exits from the main and socket is closed before the track is saved. The switch statement in register and no context used certainly looks like an issue.

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.

None yet

3 participants