zeruniverse / QQRobot

QQBot, QQ机器人(群聊小黄鸡) LINUX挂机版, SmartQQ协议。
Other
762 stars 236 forks source link

修复了因为服务器返回的html变化引起的使用问题 #5

Closed juxingzhutou closed 9 years ago

juxingzhutou commented 9 years ago

修复了获取appid、login_sig、pt_version、mibao_css的正则表达式与现在服务器返回的response中的html内容的出入。

landscape-bot commented 9 years ago

Code Health Code quality remained the same when pulling 2f813e3 on juxingzhutou:master into 327bcfe on zeruniverse:master.

zeruniverse commented 9 years ago

Merged. Thanks! It pass CI but I'll check again later

zeruniverse commented 9 years ago

https://travis-ci.org/zeruniverse/QQParking/jobs/80536056#L96

以上地址是测试服务器的测试结果(另一个项目,仍使用旧的正则表达式),旧的正则表达式仍然能正常获得参数。我刚才试了一下我这里返回格式也没有变。 能否提交下您所在地获取的HTML?

juxingzhutou commented 9 years ago

mac电源忘记带了,我周一再把html内容提交上来,我测试当日获取到的html内容确实发生了变化。

zeruniverse commented 9 years ago

谢谢!

QQ的返回格式确实存在各地区不同的问题。

hexsum commented 9 years ago

我当时看完pull request马上去测试 内容也变化了 现在又恢复原来 的确如此

zeruniverse commented 9 years ago

Then please make all different parts optional then we can adapt all versions eg: (?:a)bc matches either abc or bc

[aA]bc matches either Abc or abc

juxingzhutou commented 9 years ago

I think the version of my pull request works for both versions of html content. I will submit the html version I saw next Monday.

渣英文,凑合看看,哈哈。

hexsum commented 9 years ago

好端端的为什么要说英文。。

发自 网易邮箱大师 在2015年10月25日 10:11,HPH 写道:

I think the version of my pull request is adaptive to the two version html content. I will commit the html version I met on Monday.

渣英文,凑合看看,哈哈。

— Reply to this email directly or view it on GitHub.

zeruniverse commented 9 years ago

我刚才的电脑没输入法

On Saturday, 24 October 2015, sjdy521 notifications@github.com wrote:

好端端的为什么要说英文。。

发自 网易邮箱大师 在2015年10月25日 10:11,HPH 写道:

I think the version of my pull request is adaptive to the two version html content. I will commit the html version I met on Monday.

渣英文,凑合看看,哈哈。

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/zeruniverse/QQRobot/pull/5#issuecomment-150890174.

juxingzhutou commented 9 years ago

我只是玩玩,试试我的英文别人能不能看懂,哈哈。

我觉得我pull request里的那几个正则的写法应该是能兼容原来的html的吧。

hexsum commented 9 years ago

可以用拼音吗,哈哈 其实当前腾讯的smartqq关于appid did等参数是直接在mq.js里写死的 不去实时获取这部分参数我倒觉得关系也不大的,还可以加快登录速度

发自 网易邮箱大师 在2015年10月25日 12:06,Jeffery Zhao 写道: 我刚才的电脑没输入法

On Saturday, 24 October 2015, sjdy521 notifications@github.com wrote:

好端端的为什么要说英文。。

发自 网易邮箱大师 在2015年10月25日 10:11,HPH 写道:

I think the version of my pull request is adaptive to the two version html content. I will commit the html version I met on Monday.

渣英文,凑合看看,哈哈。

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/zeruniverse/QQRobot/pull/5#issuecomment-150890174.

— Reply to this email directly or view it on GitHub.

zeruniverse commented 9 years ago

还敢用网易邮箱……

http://www.techsite.io/2015/10/19/netease-mailbox-data-to-confirm-the-leak-affecting-million-users/

hexsum commented 9 years ago

谈到互联网安全其实就呵呵了,用什么都一样

juxingzhutou commented 9 years ago

今天返回的html又不一样了。。。今天是在=前面加了个空格,看来得写一个忽略空白符之类的兼容性比较高的方法来获取了。或者像sjdy521说的那样不要去获取这几个数据了。

juxingzhutou commented 9 years ago

我觉得其实是因为qq返回的html是会有变化的,所以我感觉我也没必要上传之前的那份html了。我觉得用正则表达式只能尽可能地兼容变化,但是要治本还是得用sjdy521说的不用正则匹配的方法靠谱。