Skip to content

fix host_del - #10

Open
thisguycodes wants to merge 1 commit into
nordnet:masterfrom
thisguycodes:thisguy.fix-host_del
Open

fix host_del#10
thisguycodes wants to merge 1 commit into
nordnet:masterfrom
thisguycodes:thisguy.fix-host_del

Conversation

@thisguycodes

@thisguycodes thisguycodes commented Aug 9, 2018

Copy link
Copy Markdown

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

ghost commented Nov 6, 2018

Copy link
Copy Markdown
    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

acalm commented Nov 6, 2018

Copy link
Copy Markdown
Contributor

@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