[GH-ISSUE #73] Unable to update ipsets #39

New issue

Closed

opened 2026-02-27 15:46:08 +03:00 by kerem · 2 comments

kerem commented

2026-02-27 15:46:08 +03:00

Owner

Originally created by @xkill on GitHub (Nov 5, 2021).
Original GitHub issue: https://github.com/proxmoxer/proxmoxer/issues/73

Hi,

I'm trying to use the library to check, add or update ipsets, but, as the IP addresses contains . it is not possible.

DOC: https://pve.proxmox.com/pve-docs/api-viewer/index.html#/cluster/firewall/ipset/{name}/{cidr}

So I have to create a call like:



from proxmoxer import ProxmoxAPI
proxmox = ProxmoxAPI(PROXMOX_HOST, user=USER_NAME, token_name=TOKEN_NAME, token_value=TOKEN_SECRET, verify_ssl=False)

CheckIP='1.2.3.4'


# Adding a IP into the blacklist IPset
proxmox.cluster.firewall.ipset.blacklist.post(cidr='1.2.3.4',comment='test asdf')

# Getting the IP from blacklist IPset
MyIP=proxmox.cluster.firewall.ipset.blacklist.get('1.2.3.4')

# MyIP = {'digest': 'randomstring',
# 'comment': 'test asdf',
# 'cidr': '1.2.3.4'}

# Updating the IP
# ????
# should be something like:

proxmox.cluster.firewall.ipset.blacklist.1.2.3.4.put(comment='new comment')
# but . is a special character in python, so it is not possible except the library changes another character to `.` on the API call.

Same problem for delete, which should be:

proxmox.cluster.firewall.ipset.blacklist.1.2.3.4.delete()

Originally created by @xkill on GitHub (Nov 5, 2021). Original GitHub issue: https://github.com/proxmoxer/proxmoxer/issues/73 Hi, I'm trying to use the library to check, add or update ipsets, but, as the IP addresses contains `.` it is not possible. DOC: [https://pve.proxmox.com/pve-docs/api-viewer/index.html#/cluster/firewall/ipset/{name}/{cidr}](https://pve.proxmox.com/pve-docs/api-viewer/index.html#/cluster/firewall/ipset/{name}/{cidr}) So I have to create a call like: ```python from proxmoxer import ProxmoxAPI proxmox = ProxmoxAPI(PROXMOX_HOST, user=USER_NAME, token_name=TOKEN_NAME, token_value=TOKEN_SECRET, verify_ssl=False) CheckIP='1.2.3.4' # Adding a IP into the blacklist IPset proxmox.cluster.firewall.ipset.blacklist.post(cidr='1.2.3.4',comment='test asdf') # Getting the IP from blacklist IPset MyIP=proxmox.cluster.firewall.ipset.blacklist.get('1.2.3.4') # MyIP = {'digest': 'randomstring', # 'comment': 'test asdf', # 'cidr': '1.2.3.4'} # Updating the IP # ???? # should be something like: proxmox.cluster.firewall.ipset.blacklist.1.2.3.4.put(comment='new comment') # but . is a special character in python, so it is not possible except the library changes another character to `.` on the API call. ``` Same problem for delete, which should be: ```python proxmox.cluster.firewall.ipset.blacklist.1.2.3.4.delete() ```

kerem

2026-02-27 15:46:08 +03:00

closed this issue
added the
type:question ❓
label

kerem commented

2026-02-27 15:46:08 +03:00

Author

Owner

@jhollowe commented on GitHub (Nov 5, 2021):

You will want to use proxmox.cluster.firewall.ipset.blacklist("1.2.3.4").put(comment='new comment')

I'm still working on the new documentation, but you can check out the Basic Usage page to learn how to fully use the dotted notation.

@jhollowe commented on GitHub (Nov 5, 2021): You will want to use `proxmox.cluster.firewall.ipset.blacklist("1.2.3.4").put(comment='new comment')` I'm still working on the new documentation, but you can check out the [Basic Usage](https://proxmoxer.github.io/docs/basic_usage/#dotted-notation) page to learn how to fully use the dotted notation.

kerem commented

2026-02-27 15:46:08 +03:00

Author

Owner

@jhollowe commented on GitHub (Nov 13, 2021):

I'm going to go ahead and close this, but if you are still having issues, please feel free to reopen or comment.

@jhollowe commented on GitHub (Nov 13, 2021): I'm going to go ahead and close this, but if you are still having issues, please feel free to reopen or comment.

kerem referenced this issue

2026-02-27 15:46:16 +03:00

[GH-ISSUE #126] PyPI package doesn't include tests #67

kerem referenced this issue

2026-02-27 15:46:40 +03:00

[PR #39] [MERGED] Exclude tests from installs #146

kerem referenced this issue

2026-02-27 15:46:43 +03:00

[PR #69] [MERGED] Version 1.2.0 #160