Change RequestLogger to lazy message evaluation

This PR uses the Logger block form that the expensive xml pretty printing is only done when needed.

Some other specs were checking for log messages via message expectations. I had to change them to output capturing