Merge "Trace Redaction - Create integration test pattern" into main
diff --git a/infra/ci/frontend/static/script.js b/infra/ci/frontend/static/script.js
index a2e9dc6..24591d4 100644
--- a/infra/ci/frontend/static/script.js
+++ b/infra/ci/frontend/static/script.js
@@ -231,7 +231,7 @@
return lastUpdateMins + ' mins ago';
if (lastUpdateMins < 60 * 24)
return Math.ceil(lastUpdateMins / 60) + ' hours ago';
- return lastUpdate.toLocaleDateString();
+ return lastUpdate.toISOString().substr(0, 10);
}
function renderCLRow(cl) {
diff --git a/src/trace_redaction/populate_allow_lists.cc b/src/trace_redaction/populate_allow_lists.cc
index f17fdcd..2ba81b6 100644
--- a/src/trace_redaction/populate_allow_lists.cc
+++ b/src/trace_redaction/populate_allow_lists.cc
@@ -29,6 +29,15 @@
return base::ErrStatus("Trace packet allow-list should be empty.");
}
+ // TRACE PACKET NOTES
+ //
+ // protos::pbzero::TracePacket::kAndroidSystemPropertyFieldNumber
+ //
+ // AndroidSystemProperty exposes a key-value pair structure with no
+ // constraints around keys or values, making fine-grain redaction
+ // difficult. Because this packet's value has no measurable, the safest
+ // option to drop the whole packet.
+
context->trace_packet_allow_list = {
protos::pbzero::TracePacket::kProcessTreeFieldNumber,
protos::pbzero::TracePacket::kProcessStatsFieldNumber,
@@ -42,7 +51,6 @@
protos::pbzero::TracePacket::kServiceEventFieldNumber,
protos::pbzero::TracePacket::kInitialDisplayStateFieldNumber,
protos::pbzero::TracePacket::kFrameTimelineEventFieldNumber,
- protos::pbzero::TracePacket::kAndroidSystemPropertyFieldNumber,
protos::pbzero::TracePacket::kSynchronizationMarkerFieldNumber,
protos::pbzero::TracePacket::kFtraceEventsFieldNumber,
