zabbix-tooling / zabbix-ldap-sync

Sync Zabbix with LDAP directory server
BSD 3-Clause "New" or "Revised" License
50 stars 36 forks source link

Typo in README | Severity attribute type #22

Closed dariodsa closed 3 years ago

dariodsa commented 4 years ago

Current

[media]
description = Email
active = 0
period = 1-5,07:00-22:00
severity = Disaster, High, Average, Warning, Information, Not Classified
onlycreate = true

Expected

[media]
description = Email
active = 0
period = 1-5,07:00-22:00
severity = 63
#Disaster, High, Average, Warning, Information, Not Classified
onlycreate = true
scoopex commented 4 years ago

Thanks for the PR. The value "Disaster, High, Average, Warning, Information, Not Classified" should be automatically converted to the numeric value. Does this not work for you?

dariodsa commented 4 years ago

No, it is working but then the documentation is wrong.

severity = 63 - Decimal value of trigger severities to send notifications about. 
scoopex commented 3 years ago

Fixed by improving the documentation: https://github.com/zabbix-tooling/zabbix-ldap-sync/commit/7935fc6e2a4075b1ae010ffd0988caab406464dc