Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Question] 关于最近有文章说WeChatMsg存在上传用户手机号等信息的说明 #492

Open
LC044 opened this issue Jul 20, 2024 · 16 comments
Labels
😇 Help Wanted Extra attention is needed

Comments

@LC044
Copy link
Owner

LC044 commented Jul 20, 2024

👌 是否检查过没有类似issue

就不检查

🧐 问题描述 | Proposed Solution

某些文章说该项目有上传用户隐私数据的行为:https://mp.weixin.qq.com/s/mvae88fxG2BFte2TTGh1ZQ

懂行的可以自己去看代码,不懂行的可以自己看截图,上传的是不是用户数据
46adaa257644d23cc4f57f90b7bdc8d1

ca4e3fabf2ed94dad38b26f141203317

1b3b002aa52af668b0e13dd1e0abf49b

ab1375e8d95ca38571cef81481fc00d2

所有数据跟弹窗保持一致,我想问问谁的手机号是:93550104,如果有的话请用这个手机号给我打个电话,没有这个偏移基址,没法解析数据,大家的微信版本用的都不一样,我得一个个去下载适配之后才能用。参考issue315, #308 #185

而且这个偏移基址所有人都一样,只跟微信版本有关

image

📝 补充信息 | Additional Information

发现这种文章欢迎大家去举报,也请各位“专家”亲自跑一下代码,看看是什么东西再说话

@sidouzi123
Copy link

我吃瓜来

个人认为版本号:昵称,账号,手机号,邮箱 这非常让人容易误会,你应该直接写明白这是手机号基址什么的,如果我只是普通用户,我认为这种误导行为就是有可能在采集我的数据,即使你不是开源,存在误导行为,别人质疑你没什么问题,接受质疑并改进也什么问题。

@LC044
Copy link
Owner Author

LC044 commented Jul 20, 2024

我吃瓜来

个人认为版本号:昵称,账号,手机号,邮箱 这非常让人容易误会,你应该直接写明白这是手机号基址什么的,如果我只是普通用户,我认为这种误导行为就是有可能在采集我的数据,即使你不是开源,存在误导行为,别人质疑你没什么问题,接受质疑并改进也什么问题。

普通用户只能看到这个:
("为了适配更多版本,需要收集微信的版本信息,该操作不会上传包括手机号、微信号、昵称等在内的任何信息\n示例数据:\n"3.9.9.27": [68065304, 0, 68065112, 0, 68066576]")

版本号:昵称,账号,手机号,邮箱这是个开发者看的,如果开发者不会自己去调试验证的话,那么我认为也就没有开发的必要了,这辈子也就告别计算机行业了

@youncyb
Copy link

youncyb commented Jul 20, 2024

我吃瓜来

个人认为版本号:昵称,账号,手机号,邮箱 这非常让人容易误会,你应该直接写明白这是手机号基址什么的,如果我只是普通用户,我认为这种误导行为就是有可能在采集我的数据,即使你不是开源,存在误导行为,别人质疑你没什么问题,接受质疑并改进也什么问题。

你有点搞笑呢,代码明明白白摆在那儿,

@bbznop
Copy link

bbznop commented Jul 20, 2024

说实话作者真为小白写了提示...

    def show_info(self):
        QMessageBox.information(self, "收集版本信息",
                                "为了适配更多版本,需要收集微信的版本信息,该操作不会上传包括手机号、微信号、昵称等在内的任何信息\n示例数据:\n\"3.9.9.27\": [68065304, 0, 68065112, 0, 68066576]"
                                )

    def upload(self, version_data):
        url = urljoin(SERVER_API_URL, 'wxBiasAddr')
        try:
            requests.post(url, json={'bias_dict': version_data})
            print('版本信息上传成功')
        except:
            pass

    def get_bias_addr(self):
        account = self.lineEdit_wx_alias.text()
        mobile = self.lineEdit_tel.text()
        name = self.lineEdit_wx_name.text()
        if not all([account, mobile, name]):
            QMessageBox.critical(self, "错误",
                                 "请把所有信息填写完整")
            return
        key = None
        db_path = "test"
        self.startBusy()
        self.thread = MyThread(account, mobile, name, key, db_path)
        self.thread.signal.connect(self.set_bias_addr)
        self.thread.start()

    def set_bias_addr(self, data):
        if self.checkBox_send_error_log.isChecked():
            self.upload(data)
        self.stopBusy()
        self.biasAddrSignal.emit(data)

@sidouzi123
Copy link

我认为这属于切实存在的 bug,我认为需要修复,如果我是普通用户我就是看不懂,你怎么办,如果这个事放在隐私保护更严重的国外,你怎么办。

正经开发流程没有这么干的。

@bbznop
Copy link

bbznop commented Jul 20, 2024

我认为这属于切实存在的 bug,我认为需要修复,如果我是普通用户我就是看不懂,你怎么办,如果这个事放在隐私保护更严重的国外,你怎么办。

正经开发流程没有这么干的。

建议“普通用户”付费使用,以获得更好的服务

@LC044
Copy link
Owner Author

LC044 commented Jul 20, 2024

我认为这属于切实存在的 bug,我认为需要修复,如果我是普通用户我就是看不懂,你怎么办,如果这个事放在隐私保护更严重的国外,你怎么办。

正经开发流程没有这么干的。

("为了适配更多版本,需要收集微信的版本信息,该操作不会上传包括手机号、微信号、昵称等在内的任何信息\n示例数据:\n"3.9.9.27": [68065304, 0, 68065112, 0, 68066576]")是这一句话看不懂吗?看不懂没关系啊,下面有个按钮,看不懂就关就行了

image

@LC044
Copy link
Owner Author

LC044 commented Jul 20, 2024

我认为这属于切实存在的 bug,我认为需要修复,如果我是普通用户我就是看不懂,你怎么办,如果这个事放在隐私保护更严重的国外,你怎么办。
正经开发流程没有这么干的。

("为了适配更多版本,需要收集微信的版本信息,该操作不会上传包括手机号、微信号、昵称等在内的任何信息\n示例数据:\n"3.9.9.27": [68065304, 0, 68065112, 0, 68066576]")是这一句话看不懂吗?看不懂没关系啊,下面有个按钮,看不懂就关就行了

image

我怀疑你连用都没用过

image

@sidouzi123
Copy link

sidouzi123 commented Jul 20, 2024

原文:你说你有理 我说我有理,你有理在源码确实没有上传,我有理在源码中确实存在手机号变量名并且这个变量名内容 upload 了,容易造成误会,我觉得还是别人评判这个问题较好。
更正:你说你有理 我说我有理,你有理在源码确实没有上传,我有理在源码中确实存在手机号字符串名(对不起我跑偏了),容易造成误会,我觉得还是别人评判这个问题较好。

@sidouzi123
Copy link

sidouzi123 commented Jul 20, 2024

原文:但是我们有办法解决两边的问题,要么改变量名 要么详细备注,推进开发规范是没问题的
更正:推进开发规范是没问题的

@LC044
Copy link
Owner Author

LC044 commented Jul 20, 2024

你说你有理 我说我有理,你有理在源码确实没有上传,我有理在源码中确实存在手机号变量名并且这个变量名内容 upload 了,容易造成误会,我觉得还是别人评判这个问题较好。

麻烦再去看一下代码,phone,account,name是干嘛用的,跟下面的upload有什么关系,只看变量名,函数调用关系你是一点都不看。中间经过了这么复杂的运算你是一点都不提,不记得有哪个开发规范说输入变量名对输出结果有影响。

@sidouzi123
Copy link

你说你有理 我说我有理,你有理在源码确实没有上传,我有理在源码中确实存在手机号变量名并且这个变量名内容 upload 了,容易造成误会,我觉得还是别人评判这个问题较好。

麻烦再去看一下代码,phone,account,name是干嘛用的,跟下面的upload有什么关系,只看变量名,函数调用关系你是一点都不看。中间经过了这么复杂的运算你是一点都不提。

哦这里我道歉,我看错了,也说错了,rdata里面变量名是没问题。但是我们最开始说的是 f.wirte的字符串您还记得吗?

@sidouzi123
Copy link

sidouzi123 commented Jul 20, 2024

那么我们就字符串名对此进行备注是否可以?因为原来的情况下在有些人审计时仍然会产生误导,即使它是写进的日志
(在此之后不做评论,首先对我中间看到了变量名跑偏了对话焦点而表示道歉,但是我仍然坚持字符串名误导是有问题的,如果您能进行备注或者修改此字符串更好,不能我也不能强求,争吵是无意义的。)
另外,质疑是可以的,事事也确实应该回音,但是直接要求删帖我认为不妥,可以要求各公众号文末或者文头更正,质疑是不能消灭的,这也是促进软件更好发展,也说明国人很注重隐私,这是好事

@Kryto614
Copy link

那么我们就字符串名对此进行备注是否可以?这仍然会产生误导,即使它是写进的日志 (在此之后不做评论,首先对我中间看到了变量名跑偏了对话焦点而表示道歉,但是我仍然坚持字符串名误导是有问题的,如果您能进行备注或者修改此字符串更好,不能我也不能强求,争吵是无意义的。)

同样觉得很有误导性。

在给团队找合适的工具,本来有付费意向的,碰到这样的隐私问题,很希望看到耐心详尽的说明。

@fdx-xdf
Copy link

fdx-xdf commented Jul 21, 2024

真是造谣一张嘴,辟谣跑断腿,那些公众号自媒体当时说窃取数据各个微信群转发,现在辟谣了只是把原来文章删了,好多人还都蒙在鼓里

@dsus-ch
Copy link

dsus-ch commented Aug 2, 2024

挺搞笑的,源码都放得的明明白白,有疑问不会调试?真不知道以后哪些倒霉客户会用这种人参与开发的软件

@wfql1024
Copy link

我认为这属于切实存在的 bug,我认为需要修复,如果我是普通用户我就是看不懂,你怎么办,如果这个事放在隐私保护更严重的国外,你怎么办。

正经开发流程没有这么干的。

我之前搞微信hook,版本不同偏移地址不同,作者需要每个版本自己下载并找寻,所以才有解密1这个直接获取。那肯定也有作者没有去弄的版本呀,如果获取不到版本信息,拿不到偏移地址,那只能通过“解密2”这个地方根据你输入的完完全全的正确的昵称和手机号在本地内存中查找,以此来获取到偏移地址。这个地址只跟版本相关,获取不到就使用不了这个软件最基础的功能。作者源码完全贴出来,有质疑想法完全可以自己拉取调试一下,而不是只会跟风,我们这些搞开发的真的对个人隐私没兴趣。作者开发个软件,能让大家免费使用最核心的功能,而且将核心部分也开源出来,我不知道这有啥可质疑的,这点我力挺作者。

@1847123212
Copy link

觉得不安全就别用啊,源代码摆在哪里还哔哔哔,如果是小白,不放心就别用,多简单得事情,人家开发者又没有对小白收费

@LC044 LC044 unpinned this issue Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
😇 Help Wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

9 participants