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

[feat] support auto login and logout at night . #35

Closed
wants to merge 1 commit into from

Conversation

WangHexie
Copy link

Ready for the new police(free internet access between 0:00 to 6:00 am) starting from Nov 11 link

Ready for the new police(free internet access between 0:00 to 6:00 am) starting from Nov 11 [link](http://xwb.neu.edu.cn/2021/1029/c5728a205587/page.htm)
@WangHexie WangHexie changed the title [feat] support auto login and logout at night automatically. [feat] support auto login and logout at night . Nov 1, 2021
@unbyte
Copy link
Member

unbyte commented Nov 2, 2021

The ipgw cli can no longer work normally due to the platform update. It needs to be adapted first, so I'll close this pr.
FYI, #36

@unbyte unbyte closed this Nov 2, 2021
@unbyte
Copy link
Member

unbyte commented Nov 2, 2021

If we're going to integrate scheduled automatic login and logout into ipgw cli, we'd better make ipgw cli a daemon and execute specific scripts containing online tasks at night.

However, I think it is best to use external job schedulers, such as cron, to provide higher freedom, and let ipgw cli continue to follow the philosophy: make each program do one thing well.

@WangHexie
Copy link
Author

Just found the tool dysfunctional. Sad.

In normal use, we have to use pc to download at night, so I think it's ok to create a screen and run it all night.
Under normal circumstances, there are not download tasks every day, so I don't think we have to treat it like a scheduled task that have to run everyday. It's better to run manually when you need.
(The daemon thing is out of my capability. Waiting for further update )

Anyway, this have to wait until the cli fixed.

@unbyte
Copy link
Member

unbyte commented Nov 2, 2021

PRs for adaption are welcomed.😍

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.

None yet

2 participants