Skip to content

Conversation

9tilov
Copy link

@9tilov 9tilov commented Sep 11, 2025

No description provided.

@9tilov 9tilov force-pushed the d9tilov branch 2 times, most recently from 44e51d0 to 2d4457b Compare September 11, 2025 15:55
Signed-off-by: ddeviatilov <9.tilov@gmail.com>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what this is about?

this.owner = owner;
}

private void startForeground() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can't exist in the library because it's meant to be reusable for other apps that aren't the official WireGuard app. We'll have to either expose a separate API to let integrators provide their own notification or just move this fully to the ui module.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree it's not the best solution but now it's quite tricky to run this VPN service on a new devices without foreground service support because Android system becomes more and more strict and can kill Vpn service.
You can read about it here.
Do we have a plan/workaround how to promote service to foreground?

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.

2 participants