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

NRPE Character Limit #32

Closed
Freeze opened this issue Oct 26, 2015 · 4 comments
Closed

NRPE Character Limit #32

Freeze opened this issue Oct 26, 2015 · 4 comments
Assignees

Comments

@Freeze
Copy link

Freeze commented Oct 26, 2015

Since our primary development platform for NRPE is GitHub, I am going to move the issue over here regarding the 1024 byte payload limit.

The initial issue was opened on the tracker here: http://tracker.nagios.org/view.php?id=564
A forum thread regarding the issue can be found here: https://support.nagios.com/forum/viewtopic.php?f=34&t=35082&p=156472&hilit=nrpe#p156472
Another highly referenced forum thread: https://support.nagios.com/forum/viewtopic.php?f=34&t=25221
A potential patch was provided here:https://web.archive.org/web/20120309192252/http://labs.opsview.com/2008/08/enhancing-nrpe-for-large-output
Another post outlining the issue: http://serverfault.com/questions/613288/truncating-return-data-as-it-is-bigger-then-nrpe-allows

There is also a branch of NRPE by abrist that patches this as well.

@tmcnag
Copy link
Contributor

tmcnag commented Oct 26, 2015

To summarize, the default packet size is 1024 bytes which is in some cases not long enough and causes status output truncation. This should either be configurable or placed so high as to be reasonably safe against overflow.

@jfrickson jfrickson self-assigned this Nov 16, 2015
@florian-asche
Copy link

+1

@jfrickson
Copy link
Contributor

Apparently, I forgot to comment on this issue. Sorry. On November 13, I made two commits. nrpe and nrpe_client will now support either the current protocol (for backward compatibility), or protocol version 3 which allows up to 64K.

This is in the nrpe-2-16-RC2 branch:
https://github.com/NagiosEnterprises/nrpe/tree/nrpe-2-16-RC2

@jfrickson
Copy link
Contributor

No problems or side-effects reported, so I'm closing the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants