Minor misc tidies

when "masking errors" are present they are constructed with a message, i think, and the name of the class is unhelpful so i think `getMessage()` is better.  for critical errors we log the full trace.