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(init): ua2f auto start #35

Merged
merged 1 commit into from
Nov 10, 2021
Merged

feat(init): ua2f auto start #35

merged 1 commit into from
Nov 10, 2021

Conversation

SunBK201
Copy link
Contributor

@SunBK201 SunBK201 commented Nov 10, 2021

ua2f auto start

@Zxilly Zxilly merged commit 0befa59 into Zxilly:master Nov 10, 2021
@1715173329
Copy link
Contributor

1715173329 commented Nov 11, 2021

However this is against with the default behavior of service files on OpenWrt.
People who want to use it should enable it by themselves.

It can be a optional feature, when someone packed it into a firmware, and share the firmware to others, you know, not everyone need it.
Also, it requires a available network connection when setting firewall rules, which is almost impossible at the first run, that I mean, when someone flashes a new firmware for their router, they actually needs to setup the network firstly before running it.

Your configurations won't be lost when you're doing a upgrade (unless you have selected "don't keep configurations" manually), so it's fine to disable it by default.

@SunBK201 SunBK201 deleted the autostart branch November 11, 2021 20:04
@Zxilly
Copy link
Owner

Zxilly commented Nov 12, 2021

@1715173329 I agree with you

Zxilly added a commit that referenced this pull request Nov 12, 2021
Zxilly added a commit that referenced this pull request Nov 12, 2021
@Zxilly
Copy link
Owner

Zxilly commented Nov 12, 2021

@SunBK201 @1715173329 I will later update readme to introduce this action to new user.

@SunBK201
Copy link
Contributor Author

agree

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.

3 participants