search

luooofan / autochaoxing

无界面超星刷课脚本，看视频+章节测试全自动，不用打开浏览器挂着
GNU General Public License v3.0
81 stars 25 forks source link

https #21

Closed RainySY closed 3 years ago

RainySY commented 3 years ago

简单修复了一下因为Https而产生

    raise InvalidURL(e, request=request)
requests.exceptions.InvalidURL: Proxy URL had no scheme, should start with http:// or https://
    raise InvalidURL(e, request=request)
requests.exceptions.InvalidURL: Proxy URL had no scheme, should start with http:// or https://

之类的报错

luooofan commented 3 years ago
  1. 无法复现
  2. 根据所示should start with http:// or https://，源代码相关links都是以http://开头的，按理来说不存在这种错误
  3. 根据对安全性的需求个人可以选择手动改为https
KevinZonda commented 3 years ago
  1. 无法复现
  2. 根据所示should start with http:// or https://，源代码相关links都是以http://开头的，按理来说不存在这种错误
  3. 根据对安全性的需求个人可以选择手动改为https

It's nonsense that still use plain HTTP in 2021.

luooofan commented 3 years ago
  1. 无法复现
  2. 根据所示should start with http:// or https://，源代码相关links都是以http://开头的，按理来说不存在这种错误
  3. 根据对安全性的需求个人可以选择手动改为https

It's nonsense that still use plain HTTP in 2021.

Hi, @KevinZonda ! You're right. It's my mistake not to take this into consideration at the begining.

But, the essential parts, for example, first visit to chaoxing website, logining, watching videos, answering questions, and so on, are all accessing with HTTPS.

The key point is this repository mainly uses selenium library to simulate learning personally rather than requests library. It means that, the mostly event happened during the program running is clicking. Therefore, how the program accessing websites by HTTP or HTTPS depends on what the website html code is. In other words, the program behaves like a person visits the websites through clicks because of the selenium library.

And if you visit chaoxing website personally, you maybe find some links will be accessed by HTTP autoly. So, maybe you should send your words to chaoxing team.

At last but not least, this pull request will be merged after my testing.

luooofan commented 3 years ago

It works well on my account in the case of good network conditions.

Thanks for your pr and comments! @RainySY @KevinZonda

KevinZonda commented 3 years ago

Cool! Cheers!

26 Apr 2021, 10:09 by @.***:

It works well on my account in the case of good network conditions.

Thanks for your pr and comments! > @RainySY https://github.com/RainySY> > @KevinZonda https://github.com/KevinZonda

— You are receiving this because you were mentioned. Reply to this email directly, > view it on GitHub https://github.com/Luoofan/autochaoxing/pull/21#issuecomment-826657255> , or > unsubscribe https://github.com/notifications/unsubscribe-auth/AH4Y5RCBPVVYQ2U37ZPAZWTTKUUWDANCNFSM42UC33MQ> .