[ZBXNEXT-3581] Drop plain text protocol, make ZBXD\1 header mandatory Created: 2016 Dec 01 Updated: 2024 Apr 10 Resolved: 2018 Apr 03 |
|
Status: | Closed |
Project: | ZABBIX FEATURE REQUESTS |
Component/s: | Agent (G), Frontend (F), Proxy (P), Server (S) |
Affects Version/s: | 3.4.0alpha1 |
Fix Version/s: | 4.0.0alpha3, 4.0 (plan) |
Type: | Change Request | Priority: | Major |
Reporter: | Glebs Ivanovskis (Inactive) | Assignee: | Vladislavs Sokurenko |
Resolution: | Fixed | Votes: | 0 |
Labels: | passive, protocols, tcp | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Issue Links: |
|
||||||||||||||||||||||||
Team: | Team A | ||||||||||||||||||||||||
Sprint: | Sprint 25, Sprint 26, Sprint 30, Sprint 31 | ||||||||||||||||||||||||
Story Points: | 0.5 |
Description |
New protocol was introduced in Zabbix 1.4, but server and proxy continued to poll passive items using old protocol and all Zabbix components continue to receive plain text messages for compatibility of newer versions with pre-1.4 agents. It turns out ( |
Comments |
Comment by Vladislavs Sokurenko [ 2018 Jan 12 ] |
Fixed in development branch: |
Comment by Andris Zeila [ 2018 Jan 29 ] |
Successfully tested. |
Comment by Vladislavs Sokurenko [ 2018 Jan 31 ] |
Fixed in:
dropped plain text protocol support and made header mandatory; fixed IP fragmentation handling by including header in Zabbix get request, Zabbix server/proxy passive check request and frontend request to Zabbix server |
Comment by richlv [ 2018 Feb 20 ] |
(7) [D] a few suggested documentation improvements :
btw, in the compatibility page, this sentence sounds strange : "Older agents from Zabbix 1.x, 2.x and previous versions of Zabbix 3.x can still be used with Zabbix 4.0" glebs.ivanovskis You can infer this information from the issue description. But would be nice to have it in Upgrade notes. Also Upgrade notes do not mention complications for self-written senders. martins-v Thanks, the two mentioned sections updated. Please review. RESOLVED glebs.ivanovskis, Can we be more specific about the 'complications' you mention? Is there anything that cannot be inferred from the information mentioned in the upgrade notes? glebs.ivanovskis I like it! Speaking of "complications", previously trappers were happily accepting messages without headers as well as messages with headers. Now they will only accept messages with protocol header. We were also thinking about giving examples of generating such header in the most popular languages... <richlv> looks good indeed, thank you.
this is a bit wrong, as there have been other incompatible config file changes before. regarding header examples in various languages, examples already abound like :
martins-v Documentation updates:
RESOLVED glebs.ivanovskis Looks good to me! I've added few snippets loosely based on sources suggested by richlv, Zabbix code and my personal hacks. I hope you are OK with wording and placement. CLOSED |
Comment by richlv [ 2018 Feb 21 ] |
this has resulted in rather confusing messages during the network discovery : |
Comment by richlv [ 2018 Dec 03 ] |
Turns out, this change is hitting a lot of users. To possibly help with search results: Message from ... is missing header. Message ignored. |