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

Issue #200 - Properly decode received log messages in GUI plugin #201

Merged
merged 1 commit into from
Mar 10, 2021

Conversation

MJJoyce
Copy link
Member

@MJJoyce MJJoyce commented Mar 10, 2021

Update the process_log_msg() helper to properly decode the received 0MQ
message from bytes into a str prior to passing it off to
log.parseSyslog. This necessary change was overlooked in our testing for
the new 0MQ send / recv call updates in Core / GUI.

Resolve #200

Update the process_log_msg() helper to properly decode the received 0MQ
message from bytes into a str prior to passing it off to
log.parseSyslog. This necessary change was overlooked in our testing for
the new 0MQ send / recv call updates in Core / GUI.

Resolve #200
@MJJoyce MJJoyce requested review from a team as code owners March 10, 2021 21:55
@MJJoyce MJJoyce merged commit 2c4a13f into master Mar 10, 2021
@MJJoyce MJJoyce deleted the issue-200 branch March 10, 2021 22:01
@nttoole
Copy link
Contributor

nttoole commented Mar 10, 2021

@MJJoyce I was hoping this was a fix to NASA-AMMOS/AIT-Core#266 but I 'spose its not.

@MJJoyce
Copy link
Member Author

MJJoyce commented Mar 11, 2021

No, different issue sadly. Or, at least, I'm not aware of it fixing that as well =D

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.

GUI errors out when processing log messages
2 participants