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

Ability to reset the monitoring data without restarting the capture #375

Open
anpic opened this issue Dec 12, 2023 · 11 comments
Open

Ability to reset the monitoring data without restarting the capture #375

anpic opened this issue Dec 12, 2023 · 11 comments
Labels
enhancement New feature or request

Comments

@anpic
Copy link

anpic commented Dec 12, 2023

Please add a button or menu item to clear the connection log at any time

@emanuele-f
Copy link
Owner

Do you mean the list in the Connections tab?

@anpic
Copy link
Author

anpic commented Dec 13, 2023

Do you mean the list in the Connections tab?

Yeap :)

@anpic
Copy link
Author

anpic commented Dec 13, 2023

Currently the list is cleared only after PCAPdroid restarting. Which is not always possible. Especially if the PCAPdroid is added to Always-on VPN.

@anpic
Copy link
Author

anpic commented Dec 13, 2023

It would be even better to add the option to delete for each item in the list via the context menu or top panel.
And even allow to select several as for the firewall blocklist and whitelist items.

@emanuele-f
Copy link
Owner

I agree on adding the ability to reset the app state without restarting it. The other option to remove individual connections seems overcomplicate to me.

@emanuele-f emanuele-f changed the title Clear logs Ability to reset the monitoring data without restarting the capture Dec 13, 2023
@emanuele-f emanuele-f added the enhancement New feature or request label Dec 13, 2023
@anpic
Copy link
Author

anpic commented Dec 13, 2023

seems overcomplicate to me.

You underestimate yourself ;)
Already your firewall is probably the best.
But without the possibility of flexible configuration, it's very difficult to use in complex scenarios.
It may be worth considering the possibility of customization through importing configs for example, in the JSON format. Even enough through intents. Without UI changes.

@emanuele-f
Copy link
Owner

You underestimate yourself ;)

Ahaha, I mean it can be complicate also for the user to have this grade of configurability. For the time being, the simplicity of a single global list it's better from my point of view, both for code complexity and user experience

@emanuele-f
Copy link
Owner

It may be worth considering the possibility of customization through importing configs for example, in the JSON format

Yes, but in part this is meant not to be simple. If people start using the app as an adblock, it could be removed from the store

@anpic
Copy link
Author

anpic commented Dec 13, 2023

and user experience

If you start conducting surveys then you will learn a lot about the needs of users ;)

it could be removed from the store

https://play.google.com/store/apps/details?id=app.greyshirts.firewall
https://play.google.com/store/apps/details?id=eu.faircode.netguard
https://play.google.com/store/apps/details?id=com.celzero.bravedns
and etc
Then it's time to remove them long ago ;)

@emanuele-f
Copy link
Owner

https://play.google.com/store/apps/details?id=eu.faircode.netguard

At least for Netguard, I know for sure that you can't do that in the google play version https://github.com/M66B/NetGuard/blob/master/ADBLOCKING.md

@anpic
Copy link
Author

anpic commented Dec 13, 2023

At least for Netguard, I know for sure that you can't do that in the google play version

It's just not necessary to loudly declare that you are adding to block ads ;)
Moreover, no one forbids adding in the form of plugins and configs. It all depends on the desire to realize or lack of it ;)

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

No branches or pull requests

2 participants