Skip to content

修复登录bug #118

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.

Sign up for GitHub

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

Jump to bottom
Merged
Samueli924 merged 2 commits into from
May 28, 2022
Merged

修复登录bug #118

Samueli924 merged 2 commits into from
May 28, 2022

Conversation

@B1gM8c
Copy link
Contributor

@B1gM8c B1gM8c commented May 27, 2022

修复登录bug

B1gM8c added 2 commits May 27, 2022 14:09
@B1gM8c
增加Des加密依赖
ad5232b 
增加Des加密依赖
@B1gM8c
新增DES算法，修复登录bug
5d566ae 
新增DES算法，修复登录bug
@B1gM8c B1gM8c mentioned this pull request May 27, 2022
Closed
@daning66
Copy link

daning66 commented May 27, 2022

IMG_20220527_151543
不行，还是有

@DaXue403
Copy link

DaXue403 commented May 27, 2022

IMG_20220527_151543 不行，还是有

这个应该不是登陆的问题了，因为已经成功获取到了课程信息；看报错和你的桌面的话，你可能用的是某IDC的服务器；超星对这些idc的ip段都进行了屏蔽，你把这个放在家用电脑上跑跑

@Samueli924
Copy link
Owner

Samueli924 commented May 27, 2022

感谢帮助反馈，今天晚上我review后若没有问题就会merge至主分支。感谢帮助

@eya46
Copy link

eya46 commented May 27, 2022 •
edited
Loading

可以把
"password": binascii.b2a_hex(secret_bytes).decode("utf-8"),
改成
"password": secret_bytes.hex(),

Samueli924
Samueli924 approved these changes May 28, 2022
View reviewed changes
Copy link
Owner

@Samueli924 Samueli924 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

检查无误。

@Samueli924 Samueli924 merged commit 56de318 into Samueli924:main May 28, 2022
@Samueli924 Samueli924 mentioned this pull request May 28, 2022
Closed
n-WN pushed a commit to n-WN/chaoxing_fork that referenced this pull request Dec 23, 2024
@Samueli924
Merge pull request Samueli924#118 from B1gM8c/main
ad59d90 
修复登录bug
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Samueli924 Samueli924 Samueli924 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@B1gM8c @daning66 @DaXue403 @Samueli924 @eya46