

<HTML><BODY><div>Squashed them. Looks like we’ve got to LGTMs here.<br><br> <blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Среда, 26 февраля 2020, 19:19 +03:00 от Igor Munkin <imun@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><style type="text/css"></style><div><div id="style_15827339980714937942_BODY">Masha,<br><br>Thanks, the patch LGTM considering Vlad's remarks and review fix on top.<br>Please don't forget to squash it with the original patch.<br><br>On 13.12.19, Maria wrote:<br>> It was possible to leak user password through setting 'replication'<br>> configuration option in first box.cfg invocation. This happened due<br>> to unconditional logging in load_cfg function. The patch introduces<br>> conditional logging.<br>><br>> Closes #4493<br>> ---<br>> Issue:<br>> <a href="https://github.com/tarantool/tarantool/issues/4493" target="_blank">https://github.com/tarantool/tarantool/issues/4493</a><br>> Branch:<br>> <a href="https://github.com/tarantool/tarantool/tree/eljashm/gh-4493-box.cfg-log-may-leak-passwords" target="_blank">https://github.com/tarantool/tarantool/tree/eljashm/gh-4493-box.cfg-log-may-leak-passwords</a><br>><br>> src/box/lua/load_cfg.lua | 3 +++<br>> test/box/load_cfg.result | 37 +++++++++++++++++++++++++++++++++++++<br>> test/box/load_cfg.test.lua | 14 ++++++++++++++<br>> test/box/lua/cfg_test6.lua | 10 ++++++++++<br>> 4 files changed, 64 insertions(+)<br>> create mode 100644 test/box/load_cfg.result<br>> create mode 100644 test/box/load_cfg.test.lua<br>> create mode 100644 test/box/lua/cfg_test6.lua<br>><br><br><snipped><br><br>> --<br>> 2.20.1 (Apple Git-117)<br>><br><br>--<br>Best regards,<br>IM</div></div></div></div></blockquote> <div> </div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Maria Khaydich</div></div></div><div> </div></div></BODY></HTML>