MissedMetadataKeyInLoggerConfig shouldn't warn if you don't use the console logger backend about credo

[mhanberg]

I can submit a patch to change the behavior if this seems appropriate.

[rrrene]

Understood. The only thing that is important is that we can not compile or execute code from the analyzed project in Credo, because it is a static analysis tool that "just looks at the code". We might already have violated that rule with the check's current implementation, which does not work for situations where --working-dir is set to some other (not loaded) Mix project.

Maybe we should just provide an example to configure the check (as proposed by @milmazz at the start of his original PR):

# uncomment an modify this to your needs:
{Credo.Check.Warning.IgnoredLoggerMetadata, [metadata_keys: "config/prod.exs" |> Config.Reader.read!() |> get_in([:logger, :console, :metadata])]},

we can do this because .credo.exs is just a regular Elixir script and if the user leverages this fact to read the project´s actual config and provides that as a parameter to the check, then we're fine.

WDYT?

[mhanberg]

I upgraded to 1.7 today and I ended up just disabling this check.

[angelikatyborska]

Maybe we should just provide an example to configure the check

That would be a good improvement. In the check's explanation? I had to configure the check in my project to make it work too, for the same reason as is the topic of this issue 🙂

In case anyone is reading this thread half-asleep, rrrene's example needs to be modified to work:

{Credo.Check.Warning.IgnoredLoggerMetadata, [metadata_keys: "config/prod.exs" |> Config.Reader.read!() |> get_in([:logger, :console, :metadata])]},

First, the check's module name is different. Second, if you hit this problem because you don't have the console backend in production, you need to pass the name of your actual logger backend. In my case it's :file_log. This config worked for me:

  {Credo.Check.Warning.MissedMetadataKeyInLoggerConfig,
   [
     metadata_keys:
       "config/prod.exs" |> Config.Reader.read!() |> get_in([:logger, :file_log, :metadata])
   ]}

[rrrene]

This is part of 1.7.2-rc.3 👍

[rrrene]

This is part of Credo 1.7.2. 👍