joy/android_packages_modules_Connectivity
Watch
1
Fork
You've already forked android_packages_modules_Connectivity
 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge "Lazy initialize CertificateTransparencyDownloader and DataStore" into main

Browse source
This commit is contained in:
Sandro Montanari 2024-12-18 06:10:50 -08:00 • committed by Gerrit Code Review
commit ff0ba88eed
2 changed files with 8 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

10
networksecurity/service/src/com/android/server/net/ct/CertificateTransparencyJob.java
View file

@ -38,6 +38,8 @@ public class CertificateTransparencyJob extends BroadcastReceiver {
private final CertificateTransparencyDownloader mCertificateTransparencyDownloader;
private final AlarmManager mAlarmManager;
private boolean mDependenciesReady = false;
/** Creates a new {@link CertificateTransparencyJob} object. */
public CertificateTransparencyJob(
Context context,
@ -50,9 +52,6 @@ public class CertificateTransparencyJob extends BroadcastReceiver {
}
void initialize() {
mDataStore.load();
mCertificateTransparencyDownloader.initialize();
mContext.registerReceiver(
this,
new IntentFilter(ConfigUpdate.ACTION_UPDATE_CT_LOGS),
@ -81,6 +80,11 @@ public class CertificateTransparencyJob extends BroadcastReceiver {
if (Config.DEBUG) {
Log.d(TAG, "Starting CT daily job.");
}
if (!mDependenciesReady) {
mDataStore.load();
mCertificateTransparencyDownloader.initialize();
mDependenciesReady = true;
}
mDataStore.setProperty(Config.CONTENT_URL, Config.URL_LOG_LIST);
mDataStore.setProperty(Config.METADATA_URL, Config.URL_SIGNATURE);

2
networksecurity/service/src/com/android/server/net/ct/CertificateTransparencyService.java
View file

@ -43,7 +43,7 @@ public class CertificateTransparencyService extends ICertificateTransparencyMana
return DeviceConfig.getBoolean(
Config.NAMESPACE_NETWORK_SECURITY,
Config.FLAG_SERVICE_ENABLED,
/* defaultValue= */ false)
/* defaultValue= */ true)
&& certificateTransparencyService()
&& certificateTransparencyConfiguration();
}