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

添加SSLVPN模式 #26

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

添加SSLVPN模式 #26

wants to merge 21 commits into from

Conversation

chenxiaoqino
Copy link
Collaborator

由于�本学期开学时还在国外,终于有动力把SSLVPN模式整理好(并测试)。可以考虑merge回master了。

使用方法:

  1. 在SSLVPN中打开academic,然后打开选课,打开课表;
  2. 正常使用即可。

需要帮助:请测试一下校内状态的导出没有Broke…

@@ -470,6 +474,29 @@ unsafeWindow.thucal=thucal=new ->
@ui.log '不支持小学期!'
return

if document.location.toString().match(/sslvpn/)
@ui.log "SSLVPN 模式: 自动登录info..."
#open info page (to utilize sslvpn system's auto-login feature)
Copy link
Owner

Choose a reason for hiding this comment

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

very clever...

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I know I'm stupid, it's for testing only at that time!

let's change that to document.location.host.match ?

Copy link
Owner

Choose a reason for hiding this comment

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

不我是说居然可以自动login,这比不用sslvpn还牛逼……

可以location.host,也可以/sslvpn/.test(不需要强制转换location)
regex本身不是问题,因为你greasemonkey header已经限制过location了

另外我当时的style太差,你看着不顺眼就修好了……
我现在的话应该会用webpack + livescript写,不过这个项目你也知道我早就不管了……

@summivox
Copy link
Owner

merge的时候别忘了把.user.js也check in,因为现在就这一个发行渠道了

@chenxiaoqino
Copy link
Collaborator Author

已经加了最新编译了。 bc965b0

@summivox
Copy link
Owner

从tuna的telegram群里面拖个还在学校的小白鼠吧,测好了自行merge好了

@chenxiaoqino
Copy link
Collaborator Author

啊,我并不是tuna成员 :-P

@summivox
Copy link
Owner

我可以拉你进那个群……

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants