added radius time mode for new event-timestamp . Format is seconds.microseconds You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/378
-- Commit Summary --
* acc_radius : added radius time mode
-- File Changes --
M modules/acc_radius/acc_radius_mod.c (23)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/378.patch https://github.com/kamailio/kamailio/pull/378.diff
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/378
Printing the float is not having any safety:
``` sprintf(smicrosec,"%f",tsecmicro); ```
Variable smicrosec has the size 18, I assume you wanted to have **10digits . 6digits**, but then enforce it because %f can have exponential representation, where the lenght can differ from system to system. Use:
``` sprintf(smicrosec,"%17.6f",tsecmicro); ```
Push this as an extra patch to your branch to show in pull request. Also, add the documentation and the PR will be merged.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/378#issuecomment-150690509
Thanks Daniel. I will handle it.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/378#issuecomment-151093637
Closed #378.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/378#event-445359750
You don't need to create a new complete path (pull request or branch) -- a second commit pushed in the same branch for this pull request will do it. Then you can have another patch to add the docs -- all will show up here if you reuse the same branch.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/378#issuecomment-151103049
Reopened #378.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/378#event-445580067
Really sorry Daniel. i spending your time this stuffs Because I am rookie for C and GitHub. i committed changes to patch-7.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/378#issuecomment-151151946
Thanks. It will be merged.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/378#issuecomment-151277496
Merged #378.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/378#event-446040936
-
Daniel-Constantin Mierla -
Yasin CANER