Two small defects in GrpcTransport:
subscribeToTask reported failures as "Failed to subscribe task push
notification config", describing the wrong operation. The message was
carried over from the old resubscribe method and renamed without being
corrected.getTaskPushNotificationConfiguration hand-rolled a null check that
throws a raw IllegalArgumentException. The check is unreachable —
GetTaskPushNotificationConfigParams already rejects a null id in its
compact constructor — and the codebase uses Assert.checkNotNullParam
for this.
No behaviour change on the happy path.
Two small defects in
GrpcTransport:subscribeToTaskreported failures as "Failed to subscribe task pushnotification config", describing the wrong operation. The message was
carried over from the old
resubscribemethod and renamed without beingcorrected.
getTaskPushNotificationConfigurationhand-rolled a null check thatthrows a raw
IllegalArgumentException. The check is unreachable —GetTaskPushNotificationConfigParamsalready rejects a nullidin itscompact constructor — and the codebase uses
Assert.checkNotNullParamfor this.
No behaviour change on the happy path.