devopsec left a comment (kamailio/kamailio#4447)
@henningw I do agree on the ACCDB_COMMENT section being removed.
I did not look into other quality of life updates in this PR but (unrelated to
this change) I do think cdrs could be more useful out of the box.
In every one of my deployments we have at a minimum 13 `cdr_extra` columns
added and typically it is more like ~20-30 extra columns of data.
It would make more sense to have some of those generalized fields in the cdrs
table by default, such as; call id, src_user, dst_user, etc..
Topic for another discussion (and how those values would be pulled from dialog
module) but I am open to helping out with the default "out of the box"
experience.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4447#issuecomment-3503055208
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4447/[email protected]>
_______________________________________________
Kamailio - Development Mailing List -- [email protected]
To unsubscribe send an email to [email protected]
Important: keep the mailing list in the recipients, do not reply only to the
sender!
