Hi, Sometime ago I submitted a rlm_digest patch at bug id 287. That patch was a fix to use MD5-Password field at FreeRadius 1.1.0. Philippe Sultan told me that this bug was already marked as 'Resolved Fixed' and "it has been decided to use a newly defined attribute Digest-HA1 rather than MD5-Password to allow a form of encrypted password storage". He also told me about bug id 332 which mentioned patch id 134. I tryed unsuccessfully to apply this patch. That is the problem. I need a patch to fix this error to FreeRadius 1.1.0. The file at CVS head could not be used cause it cannot be compiled at this version. As a temporary approach, I rewrote the original patch submitted at bug id 287 after checking some errors on it. Those errors that I found, I believe you havent notice. We have to consider two forms of storage passwords: 16 bytes string (MD5 String) or 32 bytes (MD5 Hexadecimal String) - remembering that documantion suggests to use md5sum command, which output is an hexadecimal string. Is there a solution to this problem at version 1.1.0? What about 1.1.1? Has this already been or will be fixed? Attached to this message is the patch designed to correct this problem to version 1.1.0. Best regards, EHKP (See attached file: rlm_digest.patch) --------------------------------------------------- Emerson Henrique Kfuri Pereira Divisão de Atendimento e Consultoria CECOM - Reitoria - UFMG Telefone: 34994009 ---------------------------------------------------
Emerson, you'll also find documentation here : http://wiki.freeradius.org Some testing files for rlm_digest are provided with FR 1.1.1 as well. Alan, there is a fix that should be pulled out from CVS HEAD that corrects a wrong debugging output : Line 109 in rlm_digest.c radlog(L_AUTH, "rlm_digest: Configuration item \"User-Password\" or MD5-Password is required for authentication."); Should be : radlog(L_AUTH, "rlm_digest: Configuration item \"User-Password\" or \"Digest-HA1\" is required for authentication."); Noticed during a troubleshooting session with Rafael Tonin. Philippe
"Philippe Sultan" <philippe.sultan@gmail.com> wrote:
Alan, there is a fix that should be pulled out from CVS HEAD that corrects a wrong debugging output : Line 109 in rlm_digest.c radlog(L_AUTH, "rlm_digest: Configuration item \"User-Password\" or MD5-Password is required for authentication.");
Should be : radlog(L_AUTH, "rlm_digest: Configuration item \"User-Password\" or \"Digest-HA1\" is required for authentication.");
OK, I'll go fix that. Alan DeKok.
Philippe, I would like to thank Alan and you for the attention. Sorry for the insistence. I like your software and I hope that you continue working so fine... Thank you, EHKP --------------------------------------------------- Emerson Henrique Kfuri Pereira Divisão de Atendimento e Consultoria CECOM - Reitoria - UFMG Telefone: 34994009 ---------------------------------------------------
participants (3)
-
Alan DeKok -
Emerson Henrique Kfuri Pereira -
Philippe Sultan