Skip to content

Commit 1acd87a

Browse files
jleproustzimmerle
authored andcommitted
Limit log variables to 200 characters
To avoid generating too long log lines which end up truncated (like in nginx's error log), and missing important bits.
1 parent 61c1125 commit 1acd87a

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/rule_message.cc

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ std::string RuleMessage::_details(const RuleMessage *rm) {
3232
msg.append(" [id \"" + std::to_string(rm->m_ruleId) + "\"]");
3333
msg.append(" [rev \"" + rm->m_rev + "\"]");
3434
msg.append(" [msg \"" + rm->m_message + "\"]");
35-
msg.append(" [data \"" + rm->m_data + "\"]");
35+
msg.append(" [data \"" + utils::string::limitTo(200, rm->m_data) + "\"]");
3636
msg.append(" [severity \"" +
3737
std::to_string(rm->m_severity) + "\"]");
3838
msg.append(" [ver \"" + rm->m_ver + "\"]");
@@ -43,9 +43,9 @@ std::string RuleMessage::_details(const RuleMessage *rm) {
4343
}
4444
msg.append(" [hostname \"" + std::string(rm->m_serverIpAddress) \
4545
+ "\"]");
46-
msg.append(" [uri \"" + rm->m_uriNoQueryStringDecoded + "\"]");
46+
msg.append(" [uri \"" + utils::string::limitTo(200, rm->m_uriNoQueryStringDecoded) + "\"]");
4747
msg.append(" [unique_id \"" + rm->m_id + "\"]");
48-
msg.append(" [ref \"" + rm->m_reference + "\"]");
48+
msg.append(" [ref \"" + utils::string::limitTo(200, rm->m_reference) + "\"]");
4949

5050
return msg;
5151
}
@@ -55,7 +55,7 @@ std::string RuleMessage::_errorLogTail(const RuleMessage *rm) {
5555
std::string msg;
5656

5757
msg.append("[hostname \"" + std::string(rm->m_serverIpAddress) + "\"]");
58-
msg.append(" [uri \"" + rm->m_uriNoQueryStringDecoded + "\"]");
58+
msg.append(" [uri \"" + utils::string::limitTo(200, rm->m_uriNoQueryStringDecoded) + "\"]");
5959
msg.append(" [unique_id \"" + rm->m_id + "\"]");
6060

6161
return msg;

0 commit comments

Comments
 (0)