added more log about radius result. it makes better to understand problem. i checked freeradiusclient.h and radiusclient-ng.h to have all defined variables. You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/351
-- Commit Summary --
* acc_radius : more logging for radius
-- File Changes --
M modules/acc_radius/acc_radius_mod.c (33)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/351.patch https://github.com/kamailio/kamailio/pull/351.diff
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/351
In my opinion there should be also a last 'else' branch just in case any of those library introduce a return code that you don't handle right now. You can say 'unknown response code' and print the value of the response code along with the message. You can add an additional patch for it to this pull request.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/351#issuecomment-144942198
Hmm. I missed that one, soryy for that . i added else branch. Thanks.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/351#issuecomment-144954124
Merged #351.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/351#event-425423867
-
Daniel-Constantin Mierla -
Yasin