search

unioslo / mreg-cli

Command Line Interface for Mreg
GNU General Public License v3.0
2 stars 7 forks source link

`LoginFailedError` prints "Login Failed: Login Failed" when printed to terminal #272

Closed pederhan closed 4 months ago

pederhan commented 4 months ago 
>>> from mreg_cli.exceptions import LoginFailedError
>>> exc = LoginFailedError("This is the reason we failed to login!")
>>> exc.formatted_exception()
'Login failed: Login failed'

https://github.com/unioslo/mreg-cli/blob/7485ec11c2a9ce58b4d615e85443e85e87290c5f/mreg_cli/exceptions.py#L168-L180

Specifically this line:

https://github.com/unioslo/mreg-cli/blob/7485ec11c2a9ce58b4d615e85443e85e87290c5f/mreg_cli/exceptions.py#L176

terjekv commented 4 months ago

Fixed in https://github.com/unioslo/mreg-cli/pull/277 but it wasn't automatically closed?