RE: EAP-AKA authentication support
Re: Diffs in comments. I now see the create attachment link on the bug page which I did not notice before. Re: Configure script. This is the config script for rlm_eap_aka. When did a clean build, the make complained that it could not find it so I assumed that it was needed. It has been omitted in the patch that I just attached to the bug. Re: Radius.conf, I followed what I saw in the eap-sim tests. That is the only reason I included it at all. Since eap-aka and eap-sim do very similar things and I copied most of eap-sim to implement eap-aka, I though that is was the thing to do. Now I know that is it not. Are there additional changes you need? What if anything should I do with the radius.conf file in the patch? -----Original Message----- From: freeradius-devel-bounces+bphaneuf=reefpoint.com@lists.freeradius.org [mailto:freeradius-devel-bounces+bphaneuf=reefpoint.com@lists.freeradius .org] On Behalf Of Alan DeKok Sent: Friday, September 01, 2006 1:37 PM To: FreeRadius developers mailing list Subject: Re: EAP-AKA authentication support "Phaneuf, Brian" <bphaneuf@reefpoint.com> wrote:
Alan - I have updated the bugzilla entry with the new diffs that also include the configure script in rlm_eap_aka that was missing from before.
Please don't put the diff's in the comments. Doing so makes it impossible to apply the patch. Add it as an attachment. And it's not necessary to include the 'configure' script. It can be generated from 'configure.in'. And I'm a little concerned about the whole-sale copy of radiusd.conf as an "example". Can you please add a separate attachment that is a diff of the stock radiusd.conf.in file, and the example you have. Adding an additional complete copy of radiusd.conf to the tree is a big problem. It means that now two copies have to be kept in sync, which is more work. It also means that the "example" you posted is configured for your local system, and is hard for others to use. I know there's already a copy of radiusd.conf in the src/eapsim-03 directory. That's a bad idea, and it should be fixed. Alan DeKok. -- http://deployingradius.com - The web site of the book http://deployingradius.com/blog/ - The blog - List info/subscribe/unsubscribe? See http://www.freeradius.org/list/devel.html
Hi all, That's really great that EAP-AKA is already coded, because, just like Brian, I need it too. Good job Brian ;) Since the hardest part seems done (implementation), the full integration in the freeradius official releases should be just a mather of time. Do you have any idea about how many time it could take to be released? In next 1.1.4 version maybe? Anyway, an official patch should be sooner available, right? Best Regards, Carlos Parada ________________________________ De: freeradius-devel-bounces+carlos-f-parada=ptinovacao.pt@lists.freeradius.org em nome de Phaneuf, Brian Enviada: sex 01-09-2006 18:50 Para: FreeRadius developers mailing list Assunto: RE: EAP-AKA authentication support Re: Diffs in comments. I now see the create attachment link on the bug page which I did not notice before. Re: Configure script. This is the config script for rlm_eap_aka. When did a clean build, the make complained that it could not find it so I assumed that it was needed. It has been omitted in the patch that I just attached to the bug. Re: Radius.conf, I followed what I saw in the eap-sim tests. That is the only reason I included it at all. Since eap-aka and eap-sim do very similar things and I copied most of eap-sim to implement eap-aka, I though that is was the thing to do. Now I know that is it not. Are there additional changes you need? What if anything should I do with the radius.conf file in the patch? -----Original Message----- From: freeradius-devel-bounces+bphaneuf=reefpoint.com@lists.freeradius.org [mailto:freeradius-devel-bounces+bphaneuf=reefpoint.com@lists.freeradius .org] On Behalf Of Alan DeKok Sent: Friday, September 01, 2006 1:37 PM To: FreeRadius developers mailing list Subject: Re: EAP-AKA authentication support "Phaneuf, Brian" <bphaneuf@reefpoint.com> wrote:
Alan - I have updated the bugzilla entry with the new diffs that also include the configure script in rlm_eap_aka that was missing from before.
Please don't put the diff's in the comments. Doing so makes it impossible to apply the patch. Add it as an attachment. And it's not necessary to include the 'configure' script. It can be generated from 'configure.in'. And I'm a little concerned about the whole-sale copy of radiusd.conf as an "example". Can you please add a separate attachment that is a diff of the stock radiusd.conf.in file, and the example you have. Adding an additional complete copy of radiusd.conf to the tree is a big problem. It means that now two copies have to be kept in sync, which is more work. It also means that the "example" you posted is configured for your local system, and is hard for others to use. I know there's already a copy of radiusd.conf in the src/eapsim-03 directory. That's a bad idea, and it should be fixed. Alan DeKok. -- http://deployingradius.com <http://deployingradius.com/> - The web site of the book http://deployingradius.com/blog/ - The blog - List info/subscribe/unsubscribe? See http://www.freeradius.org/list/devel.html - List info/subscribe/unsubscribe? See http://www.freeradius.org/list/devel.html
"Carlos Parada" <carlos-f-parada@ptinovacao.pt> wrote:
Do you have any idea about how many time it could take to be released? In next 1.1.4 version maybe? Anyway, an official patch should be sooner available, right?
I'm looking at integrating the changes into the 1.1 CVS branch, simply because it's easier to do there. But the code should be in 2.0, which we hope to release this fall. Alan DeKok. -- http://deployingradius.com - The web site of the book http://deployingradius.com/blog/ - The blog
participants (3)
-
Alan DeKok -
Carlos Parada -
Phaneuf, Brian