Skip to content

[Bug]: Push Configuration Config gets removed from config in rest handler #336

@RonaldMarske

Description

@RonaldMarske

What happened?

We tried to use the rest handler including push notification config. Sadly there is a normalize function, removing the push configuration from the parameters before it get's evaluated and used within the rest handler.

Parameter definition: https://github.com/a2aproject/a2a-js/blob/main/src/server/transports/rest/rest_types.ts#L87
Normalize method: https://github.com/a2aproject/a2a-js/blob/main/src/server/transports/rest/rest_transport_handler.ts#L404
Access on the property within rest handler: https://github.com/a2aproject/a2a-js/blob/main/src/server/request_handler/default_request_handler.ts#L229

This definitely looks like a bug, that it get's removed.

Relevant log output

no log output

Code of Conduct

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions