cashapp / misk

Microservice Kontainer
https://cashapp.github.io/misk/
391 stars 166 forks source link

Don't log the `UnrecognizedPropertyException` for warnings #3337

Closed mmollaverdi closed 1 day ago

mmollaverdi commented 1 week ago

It adds a fair bit of noise in tests. The error message itself is clear enough.

E.g.

15:33:59.994 [Test worker] WARN misk.config.MiskConfig -- xyz-testing.yaml: 'abc' not found in 'XyzConfig', ignoring 

com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized field "streams" (class com.squareup.cash.xyz.service.XyzConfig), not marked as ignorable (6 known properties: ...])
 at [Source: (StringReader); line: 1, column: 1674] (through reference chain: 
....