fix(eventarc): sets protojson to ignoreUnknownFields (#9807)

* fix(eventarc): sets protojson to ignoreUnknownFields

* lint

* missing import

* Reverting changes in Firestore samples

* remove unused import

Author: telpirion

eventarc/storage-handler/src/main/java/com/example/cloudrun/CloudEventController.java

@@ -46,15 +46,20 @@ ResponseEntity<String> handleCloudEvent(@RequestBody CloudEvent cloudEvent)
 
     String json = new String(cloudEvent.getData().toBytes());
     StorageObjectData.Builder builder = StorageObjectData.newBuilder();
-    JsonFormat.parser().merge(json, builder);
+
+    // If you do not ignore unknown fields, then JsonFormat.Parser returns an
+    // error when encountering a new or unknown field. Note that you might lose
+    // some event data in the unmarshaling process by ignoring unknown fields.
+    JsonFormat.Parser parser = JsonFormat.parser().ignoringUnknownFields();
+    parser.merge(json, builder);
     StorageObjectData data = builder.build();
 
     // Convert protobuf timestamp to java Instant
     Timestamp ts = data.getUpdated();
     Instant updated = Instant.ofEpochSecond(ts.getSeconds(), ts.getNanos());
     String msg =
         String.format(
-            "Cloud Storage object changed: %s/%s modified at %s\n",
+            "Cloud Storage object changed: %s/%s modified at %s%n",
             data.getBucket(), data.getName(), updated);
 
     System.out.println(msg);

functions/v2/firebase/firestore/src/main/java/functions/FirebaseFirestore.java

@@ -28,16 +28,17 @@ public class FirebaseFirestore implements CloudEventsFunction {
 
   @Override
   public void accept(CloudEvent event) throws InvalidProtocolBufferException {
-    DocumentEventData firestorEventData = DocumentEventData.parseFrom(event.getData().toBytes());
+    DocumentEventData firestoreEventData = DocumentEventData
+        .parseFrom(event.getData().toBytes());
 
     logger.info("Function triggered by event on: " + event.getSource());
     logger.info("Event type: " + event.getType());
 
     logger.info("Old value:");
-    logger.info(firestorEventData.getOldValue().toString());
+    logger.info(firestoreEventData.getOldValue().toString());
 
     logger.info("New value:");
-    logger.info(firestorEventData.getValue().toString());
+    logger.info(firestoreEventData.getValue().toString());
   }
 }
 

functions/v2/hello-gcs/src/main/java/functions/HelloGcs.java

@@ -40,7 +40,12 @@ public void accept(CloudEvent event) throws InvalidProtocolBufferException {
 
     String cloudEventData = new String(event.getData().toBytes(), StandardCharsets.UTF_8);
     StorageObjectData.Builder builder = StorageObjectData.newBuilder();
-    JsonFormat.parser().merge(cloudEventData, builder);
+
+    // If you do not ignore unknown fields, then JsonFormat.Parser returns an
+    // error when encountering a new or unknown field. Note that you might lose
+    // some event data in the unmarshaling process by ignoring unknown fields.
+    JsonFormat.Parser parser = JsonFormat.parser().ignoringUnknownFields();
+    parser.merge(cloudEventData, builder);
     StorageObjectData data = builder.build();
 
     logger.info("Bucket: " + data.getBucket());

functions/v2/imagemagick/src/main/java/functions/ImageMagick.java

@@ -156,7 +156,12 @@ private static StorageObjectData getEventData(CloudEvent event)
     // Extract Cloud Event data and convert to StorageObjectData
     String cloudEventData = new String(event.getData().toBytes(), StandardCharsets.UTF_8);
     StorageObjectData.Builder builder = StorageObjectData.newBuilder();
-    JsonFormat.parser().merge(cloudEventData, builder);
+    
+    // If you do not ignore unknown fields, then JsonFormat.Parser returns an
+    // error when encountering a new or unknown field. Note that you might lose
+    // some event data in the unmarshaling process by ignoring unknown fields.
+    JsonFormat.Parser parser = JsonFormat.parser().ignoringUnknownFields();
+    parser.merge(cloudEventData, builder);
     return builder.build();
   }
   // [START functions_imagemagick_setup]

functions/v2/ocr/ocr-process-image/src/main/java/functions/OcrProcessImage.java

@@ -71,7 +71,12 @@ public void accept(CloudEvent event) throws InvalidProtocolBufferException {
     // Unmarshal data from CloudEvent
     String cloudEventData = new String(event.getData().toBytes(), StandardCharsets.UTF_8);
     StorageObjectData.Builder builder = StorageObjectData.newBuilder();
-    JsonFormat.parser().merge(cloudEventData, builder);
+    
+    // If you do not ignore unknown fields, then JsonFormat.Parser returns an
+    // error when encountering a new or unknown field. Note that you might lose
+    // some event data in the unmarshaling process by ignoring unknown fields.
+    JsonFormat.Parser parser = JsonFormat.parser().ignoringUnknownFields();
+    parser.merge(cloudEventData, builder);
     StorageObjectData gcsEvent = builder.build();
 
     String bucket = gcsEvent.getBucket();