add per-app setting for hiding carrier/network information#340
add per-app setting for hiding carrier/network information#340maade93791 wants to merge 7 commits into
Conversation
|
I'm not sure how large the scope of this feature is, but is the intention to completely block all access to these properties? (eg. including apps that use bionic libc's |
|
@chenxiaolong its being worked on. |
maade93791
force-pushed
the
16-qpr2-carrier-info
branch
from
f7495f7 to
c319b26
Compare
|
|
||
| /** @hide */ | ||
| public class AswHideCarrierInfo extends AppSwitch { | ||
| public static final AswHideCarrierInfo I = new AswHideCarrierInfo(); |
There was a problem hiding this comment.
We should use more descriptive var names, I think
There was a problem hiding this comment.
other AppSwtich implementations use this naming scheme (e.g. AswBlockPlayIntegrityApi, AswDenyNativeDebug and so on), i went off that.
| } | ||
|
|
||
| @Override | ||
| public Boolean getImmutableValue(Context ctx, int userId, ApplicationInfo appInfo, |
There was a problem hiding this comment.
Methods that may return null should be marked with @Nullable
There was a problem hiding this comment.
base AppSwitch class doesnt declare that, didnt include it for consistency
maade93791
force-pushed
the
16-qpr2-carrier-info
branch
from
c319b26 to
9befca8
Compare
maade93791
force-pushed
the
16-qpr2-carrier-info
branch
from
9befca8 to
cf696fc
Compare
|
closed in favor of #394 |
3 participants
the current implementation handles
gsm.{operator,sim}sysprops and TelephonyManager calls that expose information for the currently installed SIM/eSIM state, its carrier and the network the connected to. also enforce CountryDetector.getCountry() to rely only on last known location and device locale, when feature is enabled.to be merged with: GrapheneOS/platform_packages_apps_Settings#425