Skip to content

[Bug]: [spec-1.0] Payload response of ListTaskPushNotificationConfig is not correct #535

@jmesnil

Description

@jmesnil

What happened?

ListTaskPushNotificationConfigResponse directly returns a ListList<TaskPushNotificationConfig> as its result
but that's not the payload defined by the spec:

// Represents a successful response for the `tasks/pushNotificationConfig/list`
// method.
message ListTaskPushNotificationConfigResponse {
  // The list of push notification configurations.
  repeated TaskPushNotificationConfig configs = 1;
  // A token, which can be sent as `page_token` to retrieve the next page.
  // If this field is omitted, there are no subsequent pages.
  string next_page_token = 2;
}

The list of configs must be wrapped in a config fields and the nextPageToken field is missing.

Relevant log output

Code of Conduct

  • I agree to follow this project's Code of Conduct

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions