[Asterisk-code-review] res_pjsip_geolocation: Initial commit (asterisk[development/16/geolocation])

George Joseph asteriskteam at digium.com
Wed Mar 9 11:59:36 CST 2022


Attention is currently required from: George Joseph.
Hello Michael Bradeen, Joshua Colp, Friendly Automation, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/c/asterisk/+/18191

to look at the new patch set (#3).

Change subject: res_pjsip_geolocation:  Initial commit
......................................................................

res_pjsip_geolocation:  Initial commit

* Added two parameters to endpoint: geoloc_incoming_call_profile
  and geoloc_outgoing_call_profile.  Alembic included.  res_pjsip
  now has a use/optional dependency on res_geolocation for
  validation.

* Added the res_pjsip_geolocation module with support for incoming
  calls.

* Fixed ast_datastore_free so it's tolerant of being passed a
  NULL pointer.  This makes it RAII_VAR compatible.

* Added PJSTR_PRINTF_SPEC and PJSTR_PRINTF_VAR(_v) convenience
  macros to res_pjsip.h.
  ast_log(LOG_ERROR, "some pjstr value: " PJSTR_PRINTF_SPEC "\n",
      PJSTR_PRINTF_VAR(&some_pjstr));

* Did more renames of functions and files.

Change-Id: I6ccb5108a5eec060efb8116502fbff3cc63d7554
---
M configs/samples/pjsip.conf.sample
A contrib/ast-db-manage/config/versions/7197536bb68d_geoloc_endpoint_params.py
M include/asterisk/res_geolocation.h
M include/asterisk/res_pjsip.h
M main/datastore.c
M res/res_geolocation.c
D res/res_geolocation/geoloc_channel.c
M res/res_geolocation/geoloc_config.c
A res/res_geolocation/geoloc_datastore.c
M res/res_geolocation/geoloc_eprofile.c
M res/res_pjsip.c
M res/res_pjsip/pjsip_config.xml
M res/res_pjsip/pjsip_configuration.c
A res/res_pjsip_geolocation.c
14 files changed, 777 insertions(+), 121 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/91/18191/3
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/18191
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: development/16/geolocation
Gerrit-Change-Id: I6ccb5108a5eec060efb8116502fbff3cc63d7554
Gerrit-Change-Number: 18191
Gerrit-PatchSet: 3
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Michael Bradeen <mbradeen at sangoma.com>
Gerrit-Attention: George Joseph <gjoseph at digium.com>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220309/e655eaad/attachment.html>


More information about the asterisk-code-review mailing list