Values of variables

[sneznaovca]

Hi,
What do you think about new option that enable log values of variables?

[shellscape]

It's not something you'd want enabled by default - that'd be asking to get in trouble by logging a lot of potential PII. I'd be up for adding an option that would add additional data to the mutate option.

[sneznaovca]

I agree that should be by default disabled. I think about new options variableValues and it just switch using Object.keys https://github.com/shellscape/apollo-log/blob/master/src/index.ts#L70
Or adding options to mutate, can looks like mutate(data:Record<string, string>, values: Record<string, string> and without override will serialize only data same as now.
But I don't like combine it. If someone will want turn it, that must doing two steps? Enable new options and modify mutate function?

[markhughes]

If this goes ahead it'd be worthwhile to add a blacklist of variables too (e.g. password or cc_number) that just replaces them with ***** or something like that.