30 Apr
2014
30 Apr
'14
1:05 p.m.
On Apr 30, 2014, at 11:50 AM, Arran Cudbard-Bell <a.cudbardb@freeradius.org> wrote:
Interesting. It's the sort of thing that could be merged into rlm_rest at some point as a sub-module.
But for now I don't see any issues in including it in v3.x.x. I do have a few comments, but they're easier made on a pull request.
I think the map section would have to change, it wouldn't be overly difficult to represent it as a set of config pairs, and you could even do nesting if you wanted a more complex structure.
Thank you, looking forward to the pull request. I do agree the ‘map’ part is a bit ugly but I wasn’t sure how to structure a config parser for an unknown set of keys. — Aaron