Merge "Add option to validate DUN networks" am: 8e2290acaf
Original change: https://android-review.googlesource.com/c/platform/frameworks/libs/net/+/2266845 Change-Id: I15b659cc9608c34d91c37adeb4e105ecb8034e8f Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
commit
b1f5ff04e8
1 changed files with 6 additions and 2 deletions
|
|
@ -16,6 +16,7 @@
|
|||
|
||||
package com.android.net.module.util;
|
||||
|
||||
import static android.net.NetworkCapabilities.NET_CAPABILITY_DUN;
|
||||
import static android.net.NetworkCapabilities.NET_CAPABILITY_INTERNET;
|
||||
import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_RESTRICTED;
|
||||
import static android.net.NetworkCapabilities.NET_CAPABILITY_NOT_VPN;
|
||||
|
|
@ -102,9 +103,12 @@ public class NetworkMonitorUtils {
|
|||
* networks.
|
||||
* @param nc Network capabilities of the network to test.
|
||||
*/
|
||||
public static boolean isValidationRequired(boolean isVpnValidationRequired,
|
||||
public static boolean isValidationRequired(boolean isDunValidationRequired,
|
||||
boolean isVpnValidationRequired,
|
||||
@NonNull final NetworkCapabilities nc) {
|
||||
// TODO: Consider requiring validation for DUN networks.
|
||||
if (isDunValidationRequired && nc.hasCapability(NET_CAPABILITY_DUN)) {
|
||||
return true;
|
||||
}
|
||||
if (!nc.hasCapability(NET_CAPABILITY_NOT_VPN)) {
|
||||
return isVpnValidationRequired;
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in a new issue