Changing "radlog" calls to RDEBUG & setting Module-Failure-Message

I run a small local patch that changes a few irritating calls to radlog into RDEBUG and sets the Module-Failure-Message inside rlm_eap; these are useful for logging per-request failure causes. See here: https://github.com/philmayers/freeradius-server/commit/baab30d0f16dc449c47a4... More generally - I personally think that radlog calls are appropriate for whole-server events (e.g. upstream proxy down) but not per-request events (except possibly very, very fatal errors) as a "radius.log" can grow surprisingly large when a client is banging away at you. What do others think?