[Freeipa-devel] [freeipa PR#546][comment] Store session cookie in a ccache option

MartinBasti freeipa-github-notification at redhat.com
Tue Mar 7 15:32:42 UTC 2017


  URL: https://github.com/freeipa/freeipa/pull/546
Title: #546: Store session cookie in a ccache option

MartinBasti commented:
"""
Pylint failed and I have a few inline comments
```
************* Module ipapython.ccache_storage
ipapython/ccache_storage.py:234: [C0305(trailing-newlines), ] Trailing newlines)
ipapython/ccache_storage.py:32: [W1612(unicode-builtin), c_text_p.from_param] unicode built-in referenced)
ipapython/ccache_storage.py:45: [E1101(no-member), c_text_p.text] Class 'value' has no 'decode' member)
ipapython/ccache_storage.py:128: [C1001(old-style-class), session_store] Old-style class defined.)
ipapython/ccache_storage.py:132: [E0710(raising-non-exception), session_store.__init__] Raising a new style class which doesn't inherit from BaseException)
ipapython/ccache_storage.py:6: [W0611(unused-import), ] Unused import os)
```
"""

See the full comment at https://github.com/freeipa/freeipa/pull/546#issuecomment-284755511


More information about the Freeipa-devel mailing list