Comment 1 for bug 1625104

Revision history for this message
Dmitry Sutyagin (dsutyagin) wrote :

I found that the issue originated in Timmy code:
commit https://github.com/adobdin/timmy/commit/a1b43e1cbd435ee2430ba347b574c578f8310130 adds usage of "connect" function and we pass all parameters, however in https://github.com/adobdin/timmy/commit/b052cede2861803c604f76913cf888e01f23dc6f we started to only provide non-default parameters to the "connect" function.

Still I believe that since connect has most parameters as non-mandatory, it should pre-populate mandatory parameters inside Client __init__ to make it work with default values when no parameters are provided from outside.