-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tested TP-link C1200 v2 #9
Comments
I forgot this part: If you have any questions or need further clarification, feel free to ask. |
Hi! |
Thank you for your response. I have tested the methods you mentioned and here are my findings: I started testing with get_status, that function works like it should. Then i tested get_firmware function, this function returend an error unknown response.
To this:
I also tested the reboot function, that works as expected, just as set wifi, that also works as expected. I also tested other functions and made necessary adjustments: get_ipv4_status get_ipv4_reservations
To this:
function get_ipv4_dhcp_leases has the same problem.
To this:
The query function works as expected. The get_full_info function also works as expected. Here is the complete edited client.py file:
Another thing to note is that i changed the _try_login function.
|
Wow. Thank you very much for your work! |
I made the pull request, I hope you can merge it. If you find any problems in my code please let me know. I can be a contributer, but I don't now how active I will be. But i will always be open for testing. |
@swwgames Thank you very much! I have merged you PR and created TplinkRouterProvider which automatically get you the right client |
I tested the new code with test.py I found that the login part worked great.
Also this router has no username, so that part of the code is not neccesary. I also tested my router in AP mode (wireless accespoint) there are some differences in the data they give, but nothing major. I hope I provided you with the right data, let me know if you need something from me. |
Thank you very much for help! |
@swwgames Could you download my updates and run test.py again? I have simplified the client |
I tested your update, and it works. The only thing is that the test file doesn't add the operation mode so some data is missing. |
@swwgames Thank you! |
Changes Made:
I've made the necessary modifications to the client.py code.
The text was updated successfully, but these errors were encountered: