IllegalArgumentException when message includes {} about kotlin-logging

[Zack-Freedman-Thoughtworks]

When I first noticed this issue, the error presented as a JsonMappingException which is likely related to using JsonLayout

[oshai]

Yes, this is really something related to the specific log4j impl. I don't have a good idea how to fix that (looking for {} placeholder in strings feels an overkill, as anyway we will just throw an exception).

[Zack-Freedman-Thoughtworks]

Is there any workaround for this or should we stay away from the version upgrade? I'd like to avoid breaking some logs for an upgrade.

[oshai]

I am not sure what you're trying to upgrade: from which version to which and how does the log call changes. I suspect using a payload is what causing this, so I expect that change to be breaking anyway, and need to be tested.

Can you please provide so more details?

[Zack-Freedman-Thoughtworks]

Hi, as you guessed, we were experimenting with a major version increment where we weren't using payloads in the past. I suppose we can still upgrade as long as we don't allow payloads, but that seems hard to enforce.