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 host_del #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ThisGuyCodes
Copy link

@ThisGuyCodes ThisGuyCodes commented Aug 9, 2018

Previously produces this error:

'error': {'code': 3005,
  'message': 'Unknown option: all',
  'name': 'OptionError'}

Works without all option. Also added explicit version (same version seen elsewhere in this library).


This change is Reviewable

Previously produces this error:
```python
'error': {'code': 3005,
  'message': 'Unknown option: all',
  'name': 'OptionError'}
```

Works without `all` option. Also added explicit version (same version seen elsewhere in this library).
@ghost
Copy link

ghost commented Nov 6, 2018

    def host_del(self, hostname):
        m = {'item': [hostname], 'method': 'host_del', 'params': {'all': True}}
        results = self.makeReq(m)

需要修改为

    def host_del(self, hostname):
        m = {'item': [hostname], 'method': 'host_del', 'params': {}}
        results = self.makeReq(m)

@acalm
Copy link
Contributor

acalm commented Nov 6, 2018

@ThisGuyCodes, sorry for the extremely delayed response, but what version of freeipa are you running?

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.

2 participants