-
Notifications
You must be signed in to change notification settings - Fork 14
Open
Labels
approvedThe topic is approved by a developerThe topic is approved by a developerenhancementAn update to an existing part of the codebaseAn update to an existing part of the codebase
Description
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:
- Consumers need to write at minimum 4 lines of boilerplate for every response, to set the protocol ID, method ID, and call ID
- 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
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
approvedThe topic is approved by a developerThe topic is approved by a developerenhancementAn update to an existing part of the codebaseAn update to an existing part of the codebase
Type
Projects
Status
Todo