[sr-dev] [kamailio/kamailio] registrar: add new xavp_rcd_mask parameter (#2168)

[Daniel-Constantin Mierla]

@lazedo - I used "random" for the personal opinions/needs of the developers which added storage of the fields in the past, not being a decision done based on a discussion devs and evaluation of the impact. If resource usage was a concern, that should have been with the first attribute added, then it was also an impact on memory usage between kamailio major versions.

The approach was to add new attributes as some dev had new needs. Because the list seemed to grow over the time, it made sense to add an option to control in long term. But from consistency point of view it should be one way or the other: store all by default and allow option to skip some; or just don't add any and allow option to specify which to store.

This new parameter it doesn't change the existing behaviour of the master branch before merging the PR -- again, at this moment in core all the attributes wanted are stored implicitly. Once merging this PR, all the attributes are still stored, so it doesn't change the default behavior. Just add option to be able to skip some via modparam.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/2168#issuecomment-563409103
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.kamailio.org/pipermail/sr-dev/attachments/20191209/903a8ea6/attachment.html>