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

[WIP] Add config memory #23

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

[WIP] Add config memory #23

wants to merge 2 commits into from

Conversation

KOWX712
Copy link
Contributor

@KOWX712 KOWX712 commented Jan 24, 2025

Changelog

  • Include all change in pr New toggle memory #19
  • Add memory for whitelist and blacklist
  • Download custom block's hosts only on host update operation, custom block hosts stored in /data/adb/Re-Malwack/cache/
  • Use parallel download, reduce host update duration

Demo

Download test package 5

To-do

  • Show list of blacklist/whitelist in WebUI
  • Whitelist hosts in repo

- Add memory for whitelist and blacklist
- Download custom block's hosts only on host update operation, custom block hosts stored in /data/adb/Re-Malwack/cache
- use parallel download, reduce host update duration
@KOWX712
Copy link
Contributor Author

KOWX712 commented Jan 24, 2025

Review

I will be abroading next week, so further work will be put on hold first.

@ZG089
Copy link
Owner

ZG089 commented Jan 24, 2025

@KOWX712 Understood. Thanks for your work, and good luck 👍

@ZG089 ZG089 self-assigned this Feb 1, 2025
@ZG089 ZG089 added enhancement New feature or request Under Review labels Feb 1, 2025
@ZG089
Copy link
Owner

ZG089 commented Feb 2, 2025

  • custom block hosts stored in /data/adb/Re-Malwack/cache/

Make it stored in /sdcard/download/Re-Malwack , better.

@KOWX712
Copy link
Contributor Author

KOWX712 commented Feb 2, 2025

Storing in place that user can access easily is not a good option, since this hosts file is a kind of cached that speed up the hosts toggle process, not an important data that we need to show to user.

@ZG089
Copy link
Owner

ZG089 commented Feb 2, 2025

@KOWX712 Ah, Okay then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Under Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants