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

Add support for multiple Event header strategies in inbound NOTIFY requests #23

Open
teles-mami opened this issue Jan 16, 2018 · 0 comments

Comments

@teles-mami
Copy link

Much like the request matching issue, a problem occurs when we want to add additional events to be processed to the existing SipPhone instance.

We cannot do this without inheriting SipPhone, which has a specific flow for processing NOTIFY requests:
validate request -> process if event is (conference | refer | presence | send bad request) -> else orphaned event error (call or transaction does not exist).

Because the middle step isso heavily cemented into the processNotify method of SipPhone, it is hard to intercept and extend this flow, and add new event packages to be processed.

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

No branches or pull requests

1 participant