Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

agents: fix profile messages format #23

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

santigimeno
Copy link
Member

In general fix ZmqAgent::send_command_message so it handles correctly both an empty and a null body.

@santigimeno santigimeno self-assigned this Nov 3, 2023
@santigimeno santigimeno force-pushed the santi/fix_zmq_cpu branch 2 times, most recently from 7f65771 to 134b185 Compare November 3, 2023 19:49
In general fix `ZmqAgent::send_command_message` so it handles correctly
both an empty and a null body.

PR-URL: #23
Reviewed-by: Trevor Norris <trev.norris@gmail.com>
@santigimeno santigimeno merged commit ffb241f into node-v20.x-nsolid-v5.x Nov 13, 2023
17 of 19 checks passed
@santigimeno santigimeno deleted the santi/fix_zmq_cpu branch November 13, 2023 11:44
trevnorris pushed a commit that referenced this pull request Dec 5, 2023
In general fix `ZmqAgent::send_command_message` so it handles correctly
both an empty and a null body.

PR-URL: #23
Reviewed-by: Trevor Norris <trev.norris@gmail.com>
trevnorris pushed a commit that referenced this pull request Dec 7, 2023
In general fix `ZmqAgent::send_command_message` so it handles correctly
both an empty and a null body.

PR-URL: #23
Reviewed-by: Trevor Norris <trev.norris@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants