[ZBX-4243] Change syslog tag not to contain a space Created: 2011 Oct 16 Updated: 2017 May 30 Resolved: 2013 Dec 19 |
|
Status: | Closed |
Project: | ZABBIX BUGS AND ISSUES |
Component/s: | Agent (G), Proxy (P), Server (S) |
Affects Version/s: | 1.8.5, 1.8.6, 1.8.7 |
Fix Version/s: | 1.8.20rc1, 2.0.11rc1, 2.2.2rc1, 2.3.0 |
Type: | Incident report | Priority: | Minor |
Reporter: | Marc | Assignee: | Unassigned |
Resolution: | Fixed | Votes: | 0 |
Labels: | logging, syslog | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified | ||
Environment: |
Linux |
Attachments: |
![]() |
||||||||||||
Issue Links: |
|
Description |
Syslog messages send by ZABBIX Server and Agent can not be parsed correctly because the syslog tag includes a blank:
That results for instance in a syslog tag "Zabbix" and a message "Agent (daemon)[32614]: zabbix_agentd listener started" ---------------------- http://www.zabbix.com/forum/showthread.php?t=16645 |
Comments |
Comment by richlv [ 2011 Nov 17 ] |
please, describe what the issue is about in the issue itself |
Comment by Oleksii Zagorskyi [ 2013 Dec 09 ] |
Could it be possible that it's by design ? Otherwise the sentence will start from "(daemon) ..." which would look not ideal, so the sentence should be improved as well. Also, first letter "Z" in "Zabbix" probably should be small "z". |
Comment by Oleksii Zagorskyi [ 2013 Dec 09 ] |
Well, looks like would be indeed good to remove/replace the space. |
Comment by Marc [ 2013 Dec 09 ] |
According to RFC5424 6.2.5. APP-NAME using 'zabbix' for all daemons might be okay. |
Comment by Andris Mednis [ 2013 Dec 09 ] |
Thanks for pointing to the RFC, Marc! Indeed, space is not allowed in APP NAME. |
Comment by Marc [ 2013 Dec 09 ] |
sounds reasonable to me. |
Comment by Andris Mednis [ 2013 Dec 13 ] |
Available in the development branch svn://svn.zabbix.com/branches/dev/ZBX-4243. |
Comment by Andris Mednis [ 2013 Dec 16 ] |
Fixed in versions pre-2.3.0 rev. 41024. |
Comment by Andris Mednis [ 2013 Dec 19 ] |
Reopened to implement the fix in all supported Zabbix versions: 1.8, 2.0, 2.2 to conform to RFC-5424. |
Comment by richlv [ 2013 Dec 19 ] |
it might be worth noting that new "app names" also include pid in square brackets, for example : zabbix_server[12345] while rfc seems to suggest using a single space to separate id, almost all other software we reviewed uses such an approach. also, some things aren't very consistent in their log messages, for example : /USR/SBIN/CRON[4007]: pam_unix(crond:session): session closed for user root /usr/sbin/cron[6716]: pam_unix(crond:session): session opened for user root by (uid=0) |
Comment by Andris Mednis [ 2013 Dec 19 ] |
Available in the development branches svn://svn.zabbix.com/branches/dev/ZBX-4243-18, svn://svn.zabbix.com/branches/dev/ZBX-4243-20, svn://svn.zabbix.com/branches/dev/ZBX-4243-22. |
Comment by Aleksandrs Saveljevs [ 2013 Dec 20 ] |
Looks good. When merging into stable branches, the original ChangeLog entry should probably be removed from 2.3.0 and added for each stable version. |
Comment by Andris Mednis [ 2013 Dec 20 ] |
Fixed in versions pre-1.8.20 rev.41126, pre-2.0.11 rev.41128, pre-2.2.2 rev.41130, pre-2.3.0 rev.41133. |
Comment by Andris Mednis [ 2013 Dec 20 ] |
Documented at |