aniketpanjwani / chomper

Internet blocker for the Linux desktop.
https://addictedto.tech/chomper/
GNU General Public License v3.0
356 stars 43 forks source link

Add missing arguments for enact_block. #21

Closed sarg closed 6 years ago

sarg commented 6 years ago

After fresh installation and issuing chomper coding 10 I get this error:

❯ ./bin/chomper coding 1     
Traceback (most recent call last):
  File "/home/sarg/devel/chomper/chomper/block.py", line 144, in <module>
    main()
  File "/home/sarg/.local/share/virtualenvs/chomper-LZFI31mM/lib/python3.6/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/home/sarg/.local/share/virtualenvs/chomper-LZFI31mM/lib/python3.6/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/home/sarg/.local/share/virtualenvs/chomper-LZFI31mM/lib/python3.6/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/sarg/.local/share/virtualenvs/chomper-LZFI31mM/lib/python3.6/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/home/sarg/devel/chomper/chomper/block.py", line 78, in main
    mitmdump_bin_path)
TypeError: enact_block() missing 2 required positional arguments: 'rule' and 'block_end_time'
CLAassistant commented 6 years ago

CLA assistant check
All committers have signed the CLA.

aniketpanjwani commented 6 years ago

Nice catch! I was doing some refactoring and must have missed this.