electric icon indicating copy to clipboard operation
electric copied to clipboard

Remove password from state inspection

Open magnetised opened this issue 1 year ago • 1 comments

Fixes #1718

magnetised avatar Sep 17 '24 09:09 magnetised

@magnetised I have a different take on the same issue - https://github.com/electric-sql/electric/pull/1772. IMO, it's better to keep the password obfuscated in memory, makes it more resilient to future code changes where we rename the connection manager module or move the connection logic into a different one. It also works in cases where the Inspect protocol is not used to print the process state, such as the case with :io.format('~p', [...]), for example.

alco avatar Oct 01 '24 00:10 alco