On Thu, 2018-04-26 at 14:08 +0100, Nick Howitt wrote:

or I have a couple of choices of adding a block:

realm realmclearos { format = suffix delimiter = "/" }

at the bottom /etc/raddb/mods-available/realm or in its own file in mods-available and symlinking to it from mods-enabled

This looks tidier to me.

There is a fair push on the mailing lists of keeping things standard and I am wondering what is the best direction - or is my solution completely wrong.

Well... there's keeping things standard, and there's making the config work. No point leaving the config "as-is" if it doesn't work for you. That's the whole point; you can change it. The thing is, don't change things that don't need changing. 99% of the default config will work for everyone, so there's no need to try and tweak it unless it's needed. So, if both of the above work for you, use whichever of the two methods you're happiest with yourself. Then check them into git or whatever so you know what you've changed from the default. You could also just comment out the "realm" call and use unlang instead, if you wanted. -- Matthew