search

TankNee / Memocast

Yet another elegant Wiz Note Client, which was built with Quasar UI Framework and based on Electron.
https://tanknee.github.io/Memocast
MIT License
362 stars 33 forks source link

建议不要在日志中不要打印明文密码 #182

Open bysunexus opened 2 years ago

bysunexus commented 2 years ago

当登录出错(网络错误)的时候,在日志中打印了明文密码

TankNee commented 2 years ago

应该是请求失败的时候打印请求日志吧？ 但是如果不打印的话有些错误比较难以排查？请教一下有什么好的办法吗？

altairwei commented 2 years ago

但是如果不打印的话有些错误比较难以排查？请教一下有什么好的办法吗？

打印 “账号或密码错误”

TankNee commented 2 years ago

应该是axios请求失败的时候报错，总不能在catch里加个判断吧😂，感觉怪怪的。

该邮件从移动设备发送

--------------原始邮件-------------- 发件人："Altair Wei @.>; 发送时间：2022年1月14日(星期五) 中午11:35 收件人："TankNee/Memocast" @.>; 抄送："TankNee @.>;"Comment @.>; 主题：Re: [TankNee/Memocast] 建议不要在日志中不要打印明文密码 (Issue #182)

但是如果不打印的话有些错误比较难以排查？请教一下有什么好的办法吗？

打印 “账号或密码错误”

— Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you commented.Message ID: @.***>