[hibernate-dev] on broken usage of logger

[Hardy Ferentschik]

Hi,

this revives my email from Friday  
(http://lists.jboss.org/pipermail/hibernate-dev/2011-November/007307.html)
where I noticed this problem as well.

Generally speaking Sanne is of course right and the right logging method  
should be used,
but does the logging framework really have to propagate the exception  
thrown by String.format?

I don't think it has to. I don't think that logging should cause an  
application to die.
Couldn't the Logger just catch the exception and write out the unformatted  
message optionally
followed by an additional warning?

--Hardy



On Wed, 23 Nov 2011 11:34:29 +0100, Sanne Grinovero <sanne at hibernate.org>  
wrote:

> Hi all,
> I'm having a chat with Hardy about how we used the Logger in some
> cases, and it turns out there's some confusion about
>
>   log.debugf   vs   log.debug
>   log.tracef     vs   log.trace
>
> So the "F" variant is needed when we want to format a message, in this
> case we should make sure the number of parameters matches exactly.
>
> Also when the string is generated, like in the case of:
>
> https://github.com/Sanne/hibernate-core/commit/3c5f8a568de99c7a6532647ab7b08c12738762ec#L36L485
>
> it's very important that we do NOT use the formatting method, but just
> a plain "log.debug( string )", otherwise it might happen at runtime
> that the dynamically generated string contains some characters which
> the String formatter might want interpret as placeholders, which are
> missing. (HHH-6817)
>
> In practice, it seems like a bad idea to allow a dynamically generated
> formatting string at all; formatting strings as defined on the Logger
> interface are safer;
>
> In case a dynamically generated message is to be generated, we should
> all make sure to pick the correct method signature; we're wondering if
> there could be a reasonable proposal to make the checks of the
> annotation processor stricter and validate for this, but there are
> some (rare) valid use cases so just throwing compilation errors would
> be nasty.
>
> Sanne
> _______________________________________________
> hibernate-dev mailing list
> hibernate-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/hibernate-dev