Pre-Submission Checklist

Commit message has the format required by CONTRIBUTING guide
Commits are split per component (core, individual modules, libs, utils, ...)
Each component has a single commit (if not, squash them into one commit)
No commits to README files for modules (changes must be done to docbook files
in doc/ subfolder, the README file is autogenerated)

Type Of Change

Small bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds new functionality)
Breaking change (fix or feature that would change existing functionality)

Checklist:

PR should be backported to stable branches
Tested changes locally
Related to issue #XXXX (replace XXXX with an open issue number)

Description

The ims_qos module had a few limitations. This fork of that module adds:

AAR for the registration (signaling path monitoring)
subsequent AARs in calls
support for STR
extra event_routes on Rx authorization session events
extra AVPs decoding and use

The dependency and use of the ims_dialog was removed. We think that the respective module has fallen behind dialog (although it should've replaced it? don't know...). Cleaning/upgrading that module too might be too much effort.

Since these changes might be breaking things for other folks using IMS, we are pushing these as an alternative module. We hope that we'll find though consensus in the future and either remove or merge with the older ims_qos module. Same would be valid for ims_dialog and dialog, yet those also might need some upgrades.

You can view, comment on, or merge this pull request online at:

https://github.com/kamailio/kamailio/pull/3921

Commit Summary

5a8111b ims_qos_npn: imported modified module
e484046 src: added ims_qos_npn to Makefile.groups

File Changes

(27 files)

M src/Makefile.groups (2)
A src/modules/ims_qos_npn/Makefile (19)
A src/modules/ims_qos_npn/README (781)
A src/modules/ims_qos_npn/cdpeventprocessor.c (382)
A src/modules/ims_qos_npn/cdpeventprocessor.h (87)
A src/modules/ims_qos_npn/doc/Makefile (4)
A src/modules/ims_qos_npn/doc/ims_qos.xml (113)
A src/modules/ims_qos_npn/doc/ims_qos_admin.xml (882)
A src/modules/ims_qos_npn/ims_qos_mod.c (2173)
A src/modules/ims_qos_npn/ims_qos_mod.h (85)
A src/modules/ims_qos_npn/ims_qos_stats.c (156)
A src/modules/ims_qos_npn/ims_qos_stats.h (38)
A src/modules/ims_qos_npn/rx_aar.c (1296)
A src/modules/ims_qos_npn/rx_aar.h (131)
A src/modules/ims_qos_npn/rx_asr.c (124)
A src/modules/ims_qos_npn/rx_asr.h (59)
A src/modules/ims_qos_npn/rx_authdata.c (494)
A src/modules/ims_qos_npn/rx_authdata.h (125)
A src/modules/ims_qos_npn/rx_avp.c (2144)
A src/modules/ims_qos_npn/rx_avp.h (118)
A src/modules/ims_qos_npn/rx_rar.c (160)
A src/modules/ims_qos_npn/rx_rar.h (34)
A src/modules/ims_qos_npn/rx_str.c (184)
A src/modules/ims_qos_npn/rx_str.h (61)
A src/modules/ims_qos_npn/sem.h (90)
A src/modules/ims_qos_npn/stats.c (79)
A src/modules/ims_qos_npn/stats.h (61)

Patch Links:

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.