Skip to content

[Enhancement]: Add RMC protocol details to response packets automatically in handlers #93

Open
@jonbarrow

Description

@jonbarrow

Checked Existing

  • I have checked the repository for duplicate issues.

What enhancement would you like to see?

In the RMC handlers, we currently take the returned RMC message and send it as-is to the client. However this has 2 main issues, both stemming from expanding the responsibility of the consumer:

  1. Consumers need to write at minimum 4 lines of boilerplate for every response, to set the protocol ID, method ID, and call ID
  2. Consumers need to manage the correct IDs themselves, which can lead to errors when the wrong IDs are set

If however we make it so that the consumer simply needs to return the RMC message and set all those IDs in the handlers here (since we have access to them already), we remove those issues entirely

Any other details to share? (OPTIONAL)

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    approvedThe topic is approved by a developerenhancementAn update to an existing part of the codebase

    Type

    No type

    Projects

    Status

    Todo

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions