Smash: Call to_log if it exists

[Attamusc]

This allows us to pass arbitrary objects into the logger and define what they should look like without requiring the caller of the log method to manually specify inclusions/exclusions.

[chr0n1x]

we're bonkers

[Attamusc]

LIVE ON THE END

[Attamusc]

Seriously though, I don't have to do this. I can just like...make another branch in the conditional

[chr0n1x]

eh it's fine

[nBerg]

I like it... :)

[nBerg]

I still dont get what it gets us though? what else would respond to this except for Hashes?

[Attamusc]

Anything. Any class that implements to_log allows for a special format. For example, say we have a user that has sensitive data that we want to pull out. Instead of writing something like

logger.info(message: "Some test", user: @user.as_json(except: [:first_name, :last_name, :other_sensitive_data]))

everytime we want to log it, we can do

class User
  def to_log
    as_json(except: [:first_name, :last_name, :other_sensitive_data])
  end
end

....

logger.info(message: "Some text", user: @user)

and consolidate that knowledge to one spot without consumers (at the logger.info level) having to know about what to sanitize, or that to_log even exists.

[nBerg]

ahh that makes sense

[chr0n1x]

👍

[bryanlatten]

@nBerg ping

[nBerg]

👍