Propagate logger metadata in spawned task

I mean it returns that if no process meta data is set, so if you always make sure to set it you should be save - which of course is more complex in a bigger data.

As an alternative you can use Logger.metadata/0 - which seems to return [] in this case and hence shouldn’t break.

iex(1)> :logger.get_process_metadata()
:undefined
iex(2)> Logger.metadata()
[]

I recently observed a strange error in some code using this pattern:

** (ArgumentError) argument error
logger.erl:621 :logger.set_process_metadata(:undefined)

I’m not familiar with Erlang but looking at the docs, it looks like it might not be technically safe because get_process_metadata can return :undefined whereas set_process_metadata only accepts a metadata map?

Thanks for the feedback!

Not really. All approaches will effectively need the same approach. However I would do it like that:

metadata = :logger.get_process_metadata()

Task.Supervisor.start_child(fn ->
  :logger.set_process_metadata(metadata)
  my_task()
end)

So the my_task/0 will not need to know anything about logger metadata.

The reason why I use :logger API instead of Logger API is that using :logger directly do not require to change map to list and then list to map and instead just uses map directly.