Skip to content

Commit

Permalink
Clean up keyring bindings (#2640)
Browse files Browse the repository at this point in the history
Remove the config file's `keyring` section and the binding in java code.
  • Loading branch information
weiminyu authored Jan 14, 2025
1 parent e4ee63b commit 4cfcc60
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 18 deletions.
6 changes: 0 additions & 6 deletions core/src/main/java/google/registry/config/RegistryConfig.java
Original file line number Diff line number Diff line change
Expand Up @@ -1099,12 +1099,6 @@ public static String provideGreetingServerId(RegistryConfigSettings config) {
return config.registryPolicy.greetingServerId;
}

@Provides
@Config("activeKeyring")
public static String provideKeyring(RegistryConfigSettings config) {
return config.keyring.activeKeyring;
}

@Provides
@Config("customLogicFactoryClass")
public static String provideCustomLogicFactoryClass(RegistryConfigSettings config) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ public class RegistryConfigSettings {
public Monitoring monitoring;
public Misc misc;
public Beam beam;
public Keyring keyring;
public RegistryTool registryTool;
public SslCertificateValidation sslCertificateValidation;
public ContactHistory contactHistory;
Expand Down Expand Up @@ -214,12 +213,6 @@ public static class Misc {
public int transientFailureRetries;
}

/** Configuration for keyrings (used to store secrets outside of source). */
// TODO(b/388835696): remove section after updating config files.
public static class Keyring {
public String activeKeyring;
}

/** Configuration options for the registry tool. */
public static class RegistryTool {
public String clientId;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -488,11 +488,6 @@ beam:
initialWorkerCount: 24
stagingBucketUrl: gcs-bucket-with-staged-templates

keyring:
# The name of the active keyring, either "Dummy" or "CSM". The latter stands
# for Cloud SecretManager.
activeKeyring: Dummy

# Configuration options relevant to the "nomulus" registry tool.
registryTool:
# OAuth client ID used by the tool.
Expand Down

0 comments on commit 4cfcc60

Please sign in to comment.