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.

Sign up for GitHub

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

Jump to bottom

Honour underlying LinkCapability #226

Open
ignoramous opened this issue Jan 14, 2021 · 3 comments
Open

Honour underlying LinkCapability #226

ignoramous opened this issue Jan 14, 2021 · 3 comments
Assignees
@hussainmohd-a
Labels
P2 Priority: 2 (important)
Milestone
v056

Comments

@ignoramous
Copy link
Collaborator

ignoramous commented Jan 14, 2021 •
edited
Loading

Underlying networks may themselves come with specific-configuration (for WIFI and CELLULAR transport types at least).

In this case, if underlying network sets a http-proxy, VPN's own http-proxy setting should be set as appropriate but let the user can override it through app's Settings.

Also, VPN needn't be set to METERED or NOT_METERED depending on the underlying active network because Android handles that for free.

See: #225
@ignoramous ignoramous added the P2 Priority: 2 (important) label Jan 14, 2021
@ignoramous ignoramous added this to the v054 milestone Jan 14, 2021
@ignoramous
Copy link
Collaborator Author

Things to reflect into VPN over from LinkProperties:

  1. Aforementioned http-proxy
  2. nat64-prefix
  3. mtu
  4. dns-servers (fixed by Allow use of default network DNS #355)
    • dns-search-domains

@ignoramous
Copy link
Collaborator Author

For nat64-prefix celzero/firestack@6c784b6

@ignoramous ignoramous modified the milestones: v054, v056 Aug 15, 2023
@ignoramous
Copy link
Collaborator Author

MTU is done, too: fa48cde, 75e493e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hussainmohd-a hussainmohd-a

Labels
P2 Priority: 2 (important)
Projects
None yet
Milestone
v056
Development

No branches or pull requests
2 participants
@ignoramous @hussainmohd-a