Query Logging: Appsignal.Ecto.log/1 errors when using the preferred Ecto.Repo loggers format about appsignal-elixir HOT 5 CLOSED

Sorry, I can't seem to reproduce this. Are you using the latest version? It seems the LogEntry that's passed in is nil..

I also don't see how this variant could cause the error, they look pretty similar. Are you doing something special? e.g. when there is another logger that does not return the entry value back, the next logger function could crash.

from appsignal-elixir.

It only happens in prod for me. I'm passing in other loggers as well so it looks like this:

loggers: [{Ecto.LogEntry, :log, []}, {MyApp.Repo.Metrics, :record_metric, []}, {Appsignal.Ecto, :log, []]

from appsignal-elixir.

You should check whether your record_metric function returns the log entry.

from appsignal-elixir.

Ah, nice catch. Didn't realize each one affected subsequent calls.

from appsignal-elixir.

NP! I also just realised this by looking at Ecto's source.

from appsignal-elixir.