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

Exempt app branch #126

Merged
merged 3 commits into from
Mar 22, 2020
Merged

Conversation

TchaikovDriver
Copy link
Contributor

Implement application-oriented bypass feature.

@wongsyrone
Copy link
Contributor

Did you save the app list to the config file?

@wongsyrone
Copy link
Contributor

I skimmed the commit without testing, looks good to me. I'm currently debugging the crash bug.

@TchaikovDriver
Copy link
Contributor Author

Did you save the app list to the config file?

No, I save the exempted app package names in SharedPreferences, which is easier to read and write. I think the size of exempted app list won't be large, so it would be better to be stored in SharedPreferences rather than in file.

@wongsyrone
Copy link
Contributor

Can you get disabled application metadata?

@wongsyrone wongsyrone merged commit dfb9da9 into trojan-gfw:master Mar 22, 2020
@TchaikovDriver TchaikovDriver deleted the exempt-app-branch branch March 31, 2020 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants