Fix typo in ABFAB PSK query
Set PSK callback only if we have an identity or a dynamic query
Don't do \n --> CR mashing in xlat.c

As it turns out, it's almost always wrong.  The correct escaping
rules are that the conf file parser reads a string, and that string
is passed verbatim through all of the layers to the final consumer.

Only the final consumer does the mashing of \n --> CR.

This means that the tests are currently broken.

%{md5:...\n} expects to get a CR on input, and it now gets a "\n".
Fixing that is the next stage.
Remove extraneous backslash
Smash \n --> CR before passing string to modules.

This is what the current code expects (see md5_xlat() ), and is
arguably correct.  An audit of the config shows there's no issue
with it.
A more complicated foreach / break test
Call fr_connection_manage() when doing fr_connection_get...()

Which means that idle connections are deleted, instead of handing
them to the caller
Typo