zydronium/google-nomulus
1
0
Fork 0
mirror of https://github.com/google/nomulus.git synced 2025-06-27 06:44:51 +02:00
Code Issues Projects Releases Packages Wiki Activity

Downgrade undeclared extension log message to INFO

Browse source 
ECatcher seems to trigger on warnings, even if no exception is thrown (?).  This warning doesn't really need to be a warning if it's WAI and not something we're actually going to bother to reach out to registrars about (in this case, 101domain apparently isn't declaring the launch extension, but we let them use it anyway).

If at some point we decide to be stricter about declaring extensions and want to go determine how often this is happening, making it INFO vs WARNING doesn't make that any harder.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=130016606
This commit is contained in:
nickfelt 2016-08-11 12:11:02 -07:00 • committed by Ben McIlwain
parent 2e88129c43
commit 72894253ae
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
java/google/registry/flows/LoggedInFlow.java
View file

@ -87,7 +87,7 @@ public abstract class LoggedInFlow extends Flow {
if (!undeclaredUrisThatError.isEmpty()) { if (!undeclaredUrisThatError.isEmpty()) {
throw new UndeclaredServiceExtensionException(undeclaredUrisThatError); throw new UndeclaredServiceExtensionException(undeclaredUrisThatError);
} else { } else {
logger.warningfmt( logger.infofmt(
"Client (%s) is attempting to run flow (%s) without declaring URIs %s on login", "Client (%s) is attempting to run flow (%s) without declaring URIs %s on login",
getClientId(), getClass().getSimpleName(), undeclaredUris); getClientId(), getClass().getSimpleName(), undeclaredUris);
} }