Re: 2.0.0 documentation for radiusd.conf

2007-06-27 Thread Alan DeKok
Hugh Messenger wrote: ... I like this new unlang. Thanks. It makes me much more confident in releasing a 2.0 that is *much* better than 1.1. Alan DeKok. - List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Re: 2.0.0 documentation for radiusd.conf

2007-06-27 Thread Hugh Messenger
Alan DeKok [EMAIL PROTECTED] said: Hugh Messenger wrote: ... I like this new unlang. Thanks. It makes me much more confident in releasing a 2.0 that is *much* better than 1.1. My only suggestion is adding some examples to the man page, and/or in the config file comments. I'm sure you

Re: 2.0.0 documentation for radiusd.conf

2007-06-27 Thread Alan DeKok
Hugh Messenger wrote: My only suggestion is adding some examples to the man page, and/or in the config file comments. I'm sure you already intend to do this, just wanted to get it back on your radar. As has been noted by others, unlang is likely to top the league table of FMF's (Frequently

Re: 2.0.0 documentation for radiusd.conf

2007-06-26 Thread Hugh Messenger
Alan Dekok [EMAIL PROTECTED] said: Hugh Messenger wrote: Any luck fixing this stuff: DEFAULT Called-Station-Id =~ ^([A-z0-9]+_[0-9]+)$, Pool-Name := `%{0}` Framed-IP-Netmask = 255.255.0.0, Fall-Through = 1 ... in either users or hints? Don't use users or hints.

Re: 2.0.0 documentation for radiusd.conf.

2007-06-15 Thread Alan Dekok
Arran Cudbard-Bell wrote: Have you managed to reproduce the bus error? No. Maybe today. Alan DeKok. -- http://deployingradius.com - The web site of the book http://deployingradius.com/blog/ - The blog - List info/subscribe/unsubscribe? See

Re: 2.0.0 documentation for radiusd.conf.

2007-06-15 Thread Alan Dekok
Arran Cudbard-Bell wrote: ... *narrowed* authorize { # Some devices send their loopback address as Nas IP Address, overwrite this with packet source. if(%{NAS-IP-Address} == 127.0.0.1){ update request { NAS-IP-Address := %{Packet-Src-IP-Address} } } Nope. It

Re: 2.0.0 documentation for radiusd.conf.

2007-06-15 Thread Arran Cudbard-Bell
Alan Dekok wrote: Arran Cudbard-Bell wrote: ... *narrowed* authorize { # Some devices send their loopback address as Nas IP Address, overwrite this with packet source. if(%{NAS-IP-Address} == 127.0.0.1){ update request { NAS-IP-Address := %{Packet-Src-IP-Address} }

Re: 2.0.0 documentation for radiusd.conf.

2007-06-15 Thread Arran Cudbard-Bell
[EMAIL PROTECTED] wrote: Hi, Ok, is there any way to get it to be more verbose about whats causing the bus error ? I'm going grab a fresh copy from the repository, just in case cvs update has mangled one of the files make distclean ./configure --with-you-options make rm -rf

Re: 2.0.0 documentation for radiusd.conf.

2007-06-15 Thread A . L . M . Buxey
Hi, Ok, is there any way to get it to be more verbose about whats causing the bus error ? I'm going grab a fresh copy from the repository, just in case cvs update has mangled one of the files make distclean ./configure --with-you-options make rm -rf /usr/local/lib/rlm_* (or

Re: 2.0.0 documentation for radiusd.conf.

2007-06-15 Thread Arran Cudbard-Bell
Arran Cudbard-Bell wrote: [EMAIL PROTECTED] wrote: Hi, Ok, is there any way to get it to be more verbose about whats causing the bus error ? I'm going grab a fresh copy from the repository, just in case cvs update has mangled one of the files make distclean ./configure

Re: 2.0.0 documentation for radiusd.conf.

2007-06-15 Thread Alan Dekok
Arran Cudbard-Bell wrote: ... modules { Module: Checking authenticate {...} for more modules to load Module: Linked to module rlm_pap Module: Instantiating pap pap { encryption_scheme = auto auto_header = yes } Module: Linked to module rlm_chap Module:

Re: 2.0.0 documentation for radiusd.conf.

2007-06-15 Thread Arran Cudbard-Bell
Alan Dekok wrote: Arran Cudbard-Bell wrote: ... modules { Module: Checking authenticate {...} for more modules to load Module: Linked to module rlm_pap Module: Instantiating pap pap { encryption_scheme = auto auto_header = yes } Module: Linked to module

Re: 2.0.0 documentation for radiusd.conf.

2007-06-14 Thread Alan Dekok
Arran Cudbard-Bell wrote: Control instead of config ? ... Both appear to work and do the same thing when updating things... I didn't want to break existing 2.0.0-pre systems. But config will be removed before 2.0.0-final. It appears that either update request is broken, or something else

Re: 2.0.0 documentation for radiusd.conf.

2007-06-14 Thread Arran Cudbard-Bell
Alan Dekok wrote: Arran Cudbard-Bell wrote: Control instead of config ? ... Both appear to work and do the same thing when updating things... I didn't want to break existing 2.0.0-pre systems. But config will be removed before 2.0.0-final. It appears that either update request is

Re: 2.0.0 documentation for radiusd.conf.

2007-06-14 Thread A . L . M . Buxey
Hi, We have three different models of NAS, each with their own weirdnesses... Apple airports send Ethernet as their medium type ... even with latest firmware? if so, nasty! HP530s Don't send a service-type in the request, they also send their loopback address as NAS-IP-Address ?! And

Re: 2.0.0 documentation for radiusd.conf.

2007-06-14 Thread Alan Dekok
Arran Cudbard-Bell wrote: ... if((%{2} == ) || (%{2} == sussex.ac.uk)){ You don't need to check if strings are empty like that. You can do: if (!%{2} || ... which may be easier to read. Oh and empty case statements screw things up in strange and weird ways... I think

Re: 2.0.0 documentation for radiusd.conf.

2007-06-14 Thread Arran Cudbard-Bell
Alan Dekok wrote: Arran Cudbard-Bell wrote: ... if((%{2} == ) || (%{2} == sussex.ac.uk)){ You don't need to check if strings are empty like that. You can do: if (!%{2} || ... which may be easier to read. Oh and empty case statements screw things up in strange and

Re: 2.0.0 documentation for radiusd.conf.

2007-06-14 Thread Arran Cudbard-Bell
rad_recv: Access-Request packet from host 139.184.6.42 port 1141, id=42, length=151 User-Name = ac221 NAS-IP-Address = 127.0.0.1 NAS-Port = 1 Called-Station-Id = 00-14-C2-B6-7D-32:eduroam Calling-Station-Id = 00-19-E3-0C-CD-58

Re: 2.0.0 documentation for radiusd.conf.

2007-06-14 Thread Arran Cudbard-Bell
expand: %{Supplicant-Flags} - 00 ??? Evaluating 00 - FALSE ?? Converting !FALSE - TRUE A string of 0 evaluates to false ? This is where you begin to need typed variables. INT(0) - FALSE INT(1) - TRUE STRING(0) - TRUE STRING(1) - TRUE -- Arran Cudbard-Bell ([EMAIL

RE:2.0.0 documentation for radiusd.conf

2007-06-14 Thread Hugh Messenger
From: Alan Dekok [EMAIL PROTECTED] Oh, and -= works. With the users file, it didn't. And there are other corner-case bugs fixed, too. Any luck fixing this stuff: DEFAULT Called-Station-Id =~ ^([A-z0-9]+_[0-9]+)$, Pool-Name := `%{0}` Framed-IP-Netmask = 255.255.0.0,

Re: 2.0.0 documentation for radiusd.conf.

2007-06-14 Thread Alan Dekok
Arran Cudbard-Bell wrote: expand: %{Supplicant-Flags} - 00 ??? Evaluating 00 - FALSE ?? Converting !FALSE - TRUE A string of 0 evaluates to false ? It's treated as an integer. This is where you begin to need typed variables. It's not a language. You can work

Re: 2.0.0 documentation for radiusd.conf

2007-06-14 Thread Alan Dekok
Hugh Messenger wrote: Any luck fixing this stuff: DEFAULT Called-Station-Id =~ ^([A-z0-9]+_[0-9]+)$, Pool-Name := `%{0}` Framed-IP-Netmask = 255.255.0.0, Fall-Through = 1 ... in either users or hints? Don't use users or hints. Use the new unlanguage. If you recall, I'm

Re: 2.0.0 documentation for radiusd.conf.

2007-06-14 Thread Arran Cudbard-Bell
Alan Dekok wrote: Arran Cudbard-Bell wrote: expand: %{Supplicant-Flags} - 00 ??? Evaluating 00 - FALSE ?? Converting !FALSE - TRUE A string of 0 evaluates to false ? It's treated as an integer. This is where you begin to need typed variables. It's not a

2.0.0 documentation for radiusd.conf.

2007-06-13 Thread Alan Dekok
I've updated the documentation for radiusd.conf, to document the new un-language. Text is attached here for comment. Alan DeKok. -- http://deployingradius.com - The web site of the book http://deployingradius.com/blog/ - The blog unlang(5) FreeRADIUS Processing

Re: 2.0.0 documentation for radiusd.conf.

2007-06-13 Thread Arran Cudbard-Bell
switch Evaluate the given string, and choose the first matching case statement inside of the current block. No statement other than case can appear in a switch block. switch string { ...

Re: 2.0.0 documentation for radiusd.conf.

2007-06-13 Thread Alan Dekok
Arran Cudbard-Bell wrote: switch These work now ? :D Yes. I just added a default to the switch statements, too. See the updated man unlang. Control instead of config ? Yes. config is already used for configuration-file stuff. Cool , very nice work :) Thanks. I think it's

Re: 2.0.0 documentation for radiusd.conf.

2007-06-13 Thread Arran Cudbard-Bell
Control instead of config ? Yes. config is already used for configuration-file stuff. Both appear to work and do the same thing when updating things... Ok, It appears that either update request is broken, or something else weird is happening. if((%{User-Name} =~ /([^-]+)-emergency-/)