fix: Check if pushNotification is the default value for gRPC in FromProto#pushNotificationConfig

fjuma · fjuma · commit d6796bcd0110 · 2025-08-20T14:04:52.000-04:00
diff --git a/spec-grpc/src/main/java/io/a2a/grpc/utils/ProtoUtils.java b/spec-grpc/src/main/java/io/a2a/grpc/utils/ProtoUtils.java
@@ -780,6 +780,9 @@ private static MessageSendConfiguration messageSendConfiguration(io.a2a.grpc.Sen
         }
 
         private static PushNotificationConfig pushNotification(io.a2a.grpc.PushNotificationConfig pushNotification, String configId) {
+            if (pushNotification == null || pushNotification.getDefaultInstanceForType().equals(pushNotification)) {
+                return null;
+            }
             return new PushNotificationConfig(
                     pushNotification.getUrl(),
                     pushNotification.getToken().isEmpty() ? null : pushNotification.getToken(),

Original file line number	Diff line number	Diff line change
`@@ -780,6 +780,9 @@ private static MessageSendConfiguration messageSendConfiguration(io.a2a.grpc.Sen`
`780`	`780`	`}`
`781`	`781`
`782`	`782`	`private static PushNotificationConfig pushNotification(io.a2a.grpc.PushNotificationConfig pushNotification, String configId) {`
	`783`	`+ if (pushNotification == null \|\| pushNotification.getDefaultInstanceForType().equals(pushNotification)) {`
	`784`	`+ return null;`
	`785`	`+ }`
`783`	`786`	`return new PushNotificationConfig(`
`784`	`787`	`pushNotification.getUrl(),`
`785`	`788`	`pushNotification.getToken().isEmpty() ? null : pushNotification.getToken(),`