Enforce no logging in production environment

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=199156367
This commit is contained in:
jianglai 2018-06-04 10:43:23 -07:00 committed by Ben McIlwain
parent 1295882307
commit 61f6e666b1
2 changed files with 23 additions and 1 deletions

View file

@ -14,6 +14,7 @@
package google.registry.proxy;
import static com.google.common.base.Preconditions.checkArgument;
import static com.google.common.base.Suppliers.memoizeWithExpiration;
import static google.registry.proxy.ProxyConfig.getProxyConfig;
import static java.util.concurrent.TimeUnit.SECONDS;
@ -79,7 +80,11 @@ public class ProxyModule {
@Parameter(names = "--env", description = "Environment to run the proxy in")
private Environment env = Environment.LOCAL;
@Parameter(names = "--log", description = "Whether to log activities for debugging")
@Parameter(
names = "--log",
description =
"Whether to log activities for debugging. "
+ "This cannot be enabled for production as logs contain PII.")
boolean log;
/**
@ -134,6 +139,9 @@ public class ProxyModule {
jCommander.usage();
throw e;
}
checkArgument(
!log || env != Environment.PRODUCTION,
"Logging cannot be enabled for production environment");
configureLogging();
return this;
}