google-nomulus/javatests/google/registry
nickfelt f2c6021db0 Split FlowReporter logging into two lines for robustness
This prevents a possible failure mode of the logging where the logged
EPP input XML is very large (which can happen e.g. for domain creates
with large SMD values).  In those cases, the XML might cause the overall
JSON string to be too large to fit within a single log entry [1], in which
case it gets split over multiple lines and breaks automatic parsing.

This mitigates that case by logging the EPP input (raw and base64-encoded)
in a separate log statement so that the more compact metadata (like clientId)
and derived values (like ICANN reporting field) will still be in an intact
JSON string even in that case, and can still be readily parsed.  It's okay
if the actual EPP XML is harder to parse, since once we're logging the right
metadata fields we shouldn't need to automatically parse the EPP XML in any
normal cases.

[1] I haven't found this exact limit or splitting algorithm, or whether it's
a property of java logging or GAE log ingestion.  The GAE logs page does note
that a single application log entry (within a request, which can have up to
1000 such entries) maxes out at 8KB, so that might be it:
https://cloud.google.com/appengine/docs/standard/java/logs/#writing_application_logs

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=153771335
2017-04-26 10:50:13 -04:00
..
backup Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:42:43 -04:00
batch Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:42:43 -04:00
bigquery Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:42:43 -04:00
builddefs Update copyright year on all license headers 2017-02-02 16:27:22 -05:00
config Remove unnecessary line to load configs 2017-02-27 11:07:40 -05:00
cron Add response information to TldFanoutAction 2017-04-10 13:32:41 -04:00
dns Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:42:43 -04:00
export Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:42:43 -04:00
flows Split FlowReporter logging into two lines for robustness 2017-04-26 10:50:13 -04:00
groups Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:47:13 -04:00
keyring/kms Refactor KmsKeyring and KmsUpdater to use a centralized serializer 2017-04-05 10:09:06 -04:00
mapreduce/inputs Update copyright year on all license headers 2017-02-02 16:27:22 -05:00
model Add Ability to return registrar contacts of a certain type 2017-04-26 10:44:14 -04:00
module Fix stale doPost references in servlet test method names 2017-02-27 11:15:01 -05:00
monitoring Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:47:13 -04:00
pricing Update copyright year on all license headers 2017-02-02 16:27:22 -05:00
rdap Remove obsolete TransferData.extendedRegistrationYears 2017-03-21 15:35:00 -04:00
rde Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:47:13 -04:00
request Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:47:13 -04:00
security Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:45:45 -04:00
server Refer to Datastore everywhere correctly by its capitalized form 2017-02-17 12:12:12 -05:00
storage/drive Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:45:45 -04:00
testing Replace deprecated Truth hasMessage() method with hasMessageThat() 2017-04-26 10:35:23 -04:00
tldconfig/idn Update copyright year on all license headers 2017-02-02 16:27:22 -05:00
tmch Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:45:45 -04:00
tools Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:45:45 -04:00
ui Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:45:45 -04:00
util Remove some unnecessary uses of MockitoJUnitRunner, which is discouraged 2017-04-26 10:42:43 -04:00
whois Don't include "Command" suffix in WHOIS metrics command name label 2017-04-10 13:46:21 -04:00
xjc Add mapper to import domains from RDE deposits 2017-02-06 16:46:46 -05:00
xml Update copyright year on all license headers 2017-02-02 16:27:22 -05:00