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

Api4com changes #1

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

Conversation

silviofernandesdeveloper
Copy link
Owner

@silviofernandesdeveloper silviofernandesdeveloper commented Aug 10, 2023

Avoid throwing an exception when we make an EVENT SIP OPTIONS request where the ruri_user variable is undefined, and with that when trying to execute the toLowerCase function it generates an exception.

// src/lwpUserAgent.js
if (config_user && ruri_user && config_user.toLowerCase() == ruri_user.toLowerCase()) {
  request.ruri.user = config_user;
}

Add a new method in lwpCall to return the list of custom headers from a call as I needed to implement a logic where calls tagged with a certain header will be answered automatically.

if (currentCall.isInProgress()) {
  const customHeaders = currentCall.getCustomHeaders()
  if (customHeaders && customHeaders['X-IntegratedCall'] === 'true') {
    return currentCall.answer()
  }
}

silviofernandesdeveloper

This comment was marked as resolved.

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.

2 participants