Those details don't belong in the error log level, that's what info or trace is for.
Trace can become so voluminous that it is switched on only on a need basis which can be too late for rare events. Also trace level as more a need to use debug tool tends to be less scrutinized for exposing sensitive data making it unsuitable for continuous operation or use in live production.
They were replying to a person who says “it’s almost always wrong for library functions to log anything”. Not just errors.