Commit report for master branch

Fri Oct 3 00:00:02 CEST 2014

New activity for FreeRADIUS (the high performance and highly configurable RADIUS server)

======
Un-tagged attributes can *never* have tags

Alan T. DeKok at 2014-10-02T23:25:12Z
Files modified:
	* src/main/tmpl.c
	* src/tests/unit/condition.txt

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/84fd8eee05022df1a6361a44894878ecc5777b57
====== 
Fix typo

Arran Cudbard-Bell at 2014-10-02T22:01:10Z
Files modified:
	* src/main/tmpl.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6b63aa612ab337c50dbc9d5cd27164862e240343
====== 
We should use strtol to parse tag and array indexes as strtoul still parses negative numbers into very large ones (which is confusing for users)

Arran Cudbard-Bell at 2014-10-02T22:01:07Z
Files modified:
	* src/main/tmpl.c
	* src/tests/unit/condition.txt

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/55b5536d686465baec6286bb90239da139344e17
====== 
They're indexes not references...

Arran Cudbard-Bell at 2014-10-02T22:01:04Z
Files modified:
	* src/main/xlat.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/7ffc1bf52c5f4fcf270425f41c81404650ef8382
====== 
Formatting

Arran Cudbard-Bell at 2014-10-02T22:01:01Z
Files modified:
	* src/modules/rlm_files/rlm_files.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/85e8e662976a6376d4e01e7caee64def0403f736
====== 
Fix invalid request/list errors

Arran Cudbard-Bell at 2014-10-02T22:00:57Z
Files modified:
	* src/main/tmpl.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a86a33dc8ae69da491fb4a5f8dbe132830c02279
====== 
Indent condition error markers

Arran Cudbard-Bell at 2014-10-02T22:00:54Z
Files modified:
	* src/main/conffile.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/94cb99aaee5f662ceae5510c38263904dfcd626d
====== 
I hate doxygen

Arran Cudbard-Bell at 2014-10-02T20:59:51Z
Files modified:
	* doc/source/Doxyfile

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/0f7e22ef70488277c7f73113c587190274211cd0
====== 
Convert the majority of tmpl functions to use a similar signature as the parse functions

This means condition parsing error markers now extend into attribute references

Arran Cudbard-Bell at 2014-10-02T20:48:49Z
Files modified:
	* src/include/parser.h
	* src/include/tmpl.h
	* src/main/map.c
	* src/main/modcall.c
	* src/main/parser.c
	* src/main/tmpl.c
	* src/main/valuepair.c
	* src/main/xlat.c
	* src/modules/rlm_expr/rlm_expr.c
	* src/modules/rlm_rest/rest.c
	* src/tests/unit/condition.txt

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c1411971634067c3f754993a7834f8f7fc0f2870
====== 
Move fr_canonicalize_error to log.c

I still don't like the name, partly because now I have to remember how to spell canonicalize, and it seems like it should have double n's to me

Arran Cudbard-Bell at 2014-10-02T20:48:46Z
Files modified:
	* src/include/log.h
	* src/main/log.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b03e6feda09eb2e012377130bfeda09d6cdbcfce
====== 
Convert tmpl_afrom_attr_substr to use a similar signature as other parsing functions

Arran Cudbard-Bell at 2014-10-02T16:20:29Z
Files modified:
	* src/include/tmpl.h
	* src/main/map.c
	* src/main/tmpl.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/ed458b013221ed91ca22c6a577c6c7b039251c0b
====== 
Add comment for fr_canonicalize_error

Arran Cudbard-Bell at 2014-10-02T16:20:25Z
Files modified:
	* src/main/parser.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f09d7deeb3b161ffcc61fa0c0abf9bfa20ca117b
====== 
Output variables come first

Arran Cudbard-Bell at 2014-10-02T16:20:22Z
Files modified:
	* src/include/parser.h
	* src/main/conffile.c
	* src/main/modcall.c
	* src/main/parser.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/96b5fc82ab5647a63435b06ff6948cc7fbdc3ff1
====== 
Tests for commas, too

Alan T. DeKok at 2014-10-02T15:35:50Z
Files modified:
	* src/tests/keywords/comments

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/1e14bc34a9618ffff309b559f17fce01a948d2ad
====== 
Stop on comma and EOL, too

Alan T. DeKok at 2014-10-02T15:34:51Z
Files modified:
	* src/main/conffile.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/26ce727c2616fd8d8cf88223cf36b2da7e485a8e
====== 
Fix comments after module reference.

	ok # foo!

Alan T. DeKok at 2014-10-02T15:34:50Z
Files modified:
	* src/main/conffile.c
	* src/tests/keywords/comments

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2615fdf39916f4b6b08a2af58679b4fbfa98738e
====== 
Add test for comments

Arran Cudbard-Bell at 2014-10-02T13:54:22Z
Files modified:
	* src/tests/keywords/comments

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/cf151d0567a8c455f96713a66df445adcc2bb579
====== 
Fixup rlm_sqlcounter warnings

Arran Cudbard-Bell at 2014-10-02T13:38:18Z
Files modified:
	* src/modules/rlm_sqlcounter/rlm_sqlcounter.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/578b222d7bbccc4d41d72dddd2ecab42fec5f7f9
====== 
Hint that request and cmd should never be NULL in radius_exec_program

Arran Cudbard-Bell at 2014-10-02T11:47:36Z
Files modified:
	* src/include/radiusd.h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d98b93bb3f4f330e21deca760cc8c4a81dd419cc
====== 
Change logging in radius_exec_program to use R* macros

RERROR should always be used where an error may indicate a failure of an external resource like a script, or database and we have the REQUEST available.

It outputs to the main server log like ERROR but also includes the request number, so multiple errors can be tied together when inspecting the log.

Arran Cudbard-Bell at 2014-10-02T11:44:07Z
Files modified:
	* src/main/exec.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6fe7087e89379df88c83209776a41eaea139fecb
====== 
Change eap_vp2packet to use fr_strerror_printf to output its errors

Because "Malformed EAP message" as an error without any more detail is not useful

Arran Cudbard-Bell at 2014-10-02T11:40:25Z
Files modified:
	* src/modules/rlm_eap/libeap/eapcommon.c
	* src/modules/rlm_eap/libeap/eapsimlib.c
	* src/modules/rlm_eap/radeapclient.c
	* src/modules/rlm_eap/rlm_eap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/cbe39df5b7c4941881d6109eb59468a3d9cc29fe
====== 
Whitespace

Arran Cudbard-Bell at 2014-10-02T11:39:26Z
Files modified:
	* src/modules/rlm_eap/rlm_eap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a44a92473a480a6bb46ebd0a320996e8e88fb29e
====== 
More verbose error message on proxy failure. Including the User-Name the request was for, is extremely useful for eduroam.

Arran Cudbard-Bell at 2014-10-02T11:32:08Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a4474b0c40b0ac9390e97cea490940b95b541e38
====== 
-- 
This commit summary was generated @2014-10-03T00:00:02Z by lgfeed version 0.00 (https://github.com/arr2036/lgfeed).