-
Notifications
You must be signed in to change notification settings - Fork 1
ipaddress parameter is unused in Community.remove_host #3
Copy link
Copy link
Open
Labels
bugSomething isn't workingSomething isn't working
Description
A params dict is constructed, but never used.
mreg-api/mreg_api/models/models.py
Lines 1874 to 1892 in 70c71fc
| def remove_host(self, host: Host, ipaddress: IP_AddressT | None) -> bool: | |
| """Remove a host from the community. | |
| :param host: The host to remove. | |
| :returns: True if the host was removed, False otherwise. | |
| """ | |
| from mreg_api.client import MregClient # noqa: PLC0415 | |
| params: QueryParams = {} | |
| if ipaddress: | |
| params["ipaddress"] = str(ipaddress) | |
| resp = MregClient().delete( | |
| Endpoint.NetworkCommunityHost.with_params( | |
| self.network_address, | |
| self.id, | |
| host.id, | |
| ) | |
| ) | |
| return resp.is_success if resp else False |
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working