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

fix login for zabbix >=5.4 #37

Closed
wants to merge 1 commit into from
Closed

Conversation

ghormoon
Copy link

@ghormoon ghormoon commented Oct 5, 2022

Since version 5.4, zabbix changed 'user' parameter name for 'username' in login method. Handling added to support versions >=5.4

zabbix/zabbix_api.py Outdated Show resolved Hide resolved
@ghormoon
Copy link
Author

ghormoon commented Nov 6, 2022

mr updated with packaging.version.parse instead

@NoxInmortus
Copy link

If this fix zabbix-api for Zabbix 6.2 I would be glad!

@gescheit
Copy link
Owner

gescheit commented Dec 9, 2022

packaging is included in python:

>>> import packaging
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'packaging'

https://stackoverflow.com/a/11887825 this implementation is good enough for current problem.

@chrisallennc
Copy link

chrisallennc commented Apr 12, 2023

Is there an ETA for when this will be merged? With Zabbix 6.4 the "user" parameter has been removed and "username" must be used so this module cannot be used with 6.4+ https://www.zabbix.com/documentation/current/en/manual/api/changes#user

@rjorgenson-ns
Copy link

Also needing this update for an imminent upgrade to zabbix 6.4

@eliottcassidy2000
Copy link

+1, would be great to have this merged so we can use it with zabbix 6.4

@gescheit
Copy link
Owner

Fixed in #39

@gescheit gescheit closed this Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants