On Mon, Feb 17, 2014 at 10:32:50AM -0500, Alan DeKok wrote:
Please test.
Basic tests seem OK - builds as a debian package on wheezy/amd64, and works. Haven't exercised much else though. There's a warning about potentially unused variable compare in src/lib/valuepair.c:2651, but it can't happen looking at the code. Maybe needs a compare=compare to silence it?
One major cosmetic change is the hiding of secret keys when using "radiusd -X" ... They still show up when using "radiusd -Xx".
Seems reasonable - agree that User-Password should continue to show. Need enough to show that people have the wrong shared secret (corrupt user-password) or the user-password is wrong, which I think this will do. I don't think that it's a good thing to start mangling or blanking out attributes in debug output.
If anyone objects, please let me know.
Looks OK as is now. Just need to stop people posting -Xx by default, which a reasonable number still seem to do :( Matthew -- Matthew Newton, Ph.D. <mcn4@le.ac.uk> Systems Specialist, Infrastructure Services, I.T. Services, University of Leicester, Leicester LE1 7RH, United Kingdom For IT help contact helpdesk extn. 2253, <ithelp@le.ac.uk>