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

Revert logging in server_io_callback #46

Merged
merged 1 commit into from Apr 24, 2020
Merged

Revert logging in server_io_callback #46

merged 1 commit into from Apr 24, 2020

Conversation

danrodrig
Copy link
Contributor

Buffer was used directly in g_message call, but there is no guarantee
that it's going to be null terminated.

Use logging as before removal of libssh.

Fixes #45

@danrodrig danrodrig added this to the 0.2.1 milestone Apr 23, 2020
Buffer was used directly in g_message call, but there is no guarantee
that it's going to be null terminated.

Use logging as before removal of libssh.
@danrodrig danrodrig changed the title Revert logging in servce_io_callback Revert logging in server_io_callback Apr 23, 2020
@danrodrig
Copy link
Contributor Author

danrodrig commented Apr 23, 2020

/test /packit build 👀

Copy link
Contributor

@StykMartin StykMartin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverted code from ed9ce41.

@danrodrig danrodrig requested review from renanrodrigo, a team and cbouchar and removed request for a team April 23, 2020 14:41
Copy link
Contributor

@cbouchar cbouchar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe you are asking to replace g_message back to fwrite. I'm good with that.

@danrodrig danrodrig merged commit ae2a427 into restraint-harness:master Apr 24, 2020
@danrodrig danrodrig deleted the restore-io-server-logging branch April 24, 2020 07:04
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.

Restraint daemon logs incorrect data
3 participants