search

atlasmap-attic / atlasmap-runtime

AtlasMap runtime engine and design time services
http://atlasmap.io
Other
3 stars 8 forks source link

Make sure using Audit for user level log at runtime, don't log #271

Open apupier opened 6 years ago

apupier commented 6 years ago

there are several places where a String.format is duplicated and used first to log and then to audit. It would be better to factorize code and calculate the String.format a single time.

it will avoid to hit this kind of issue: https://github.com/atlasmap/atlasmap-runtime/issues/270

igarashitm commented 6 years ago

IMO atlasmap runtime itself shouldn't emit any user level log other than for debug purpose, instead collect Validation at design time and Audit at runtime and let invoker choose what to do with those.

mattrpav commented 6 years ago

@igarashitm there is a ticket (#7) addressing this very topic that was never fully addressed due to time constraints.