Skip to content

fix: fix authentication failures #37

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mig8447
Copy link

@mig8447 mig8447 commented May 30, 2021

allow subsequent requests to use the same authentication instead of re-authenticating all the time

allow subsequent requests to use the same authentication instead of re-authenticating all the time
@mig8447
Copy link
Author

mig8447 commented May 30, 2021

Control commands didn't work either with firmware 1.26

@LinuxChristian
Copy link
Owner

@mig8447, thanks for your PR and I am really sorry for not getting back to you sooner. I never got notifications from GH about the PR. Likely because you didn't add me as a reviewer 🤔 You should just have pinged me when I didn't respond.

I will take a look and get back to you soon. Hopefully within a week. I don't really use the plug myself anymore so it just takes a bit of time to ensure everything still works.

@LinuxChristian LinuxChristian self-requested a review December 1, 2021 08:36
@LinuxChristian
Copy link
Owner

I am very sorry for taking so long to get on with this - I really appriciate the contribution

To anyone reading this in the future - I don't have access to my plugs anymore so I cannot ensure this PR will not break support for older plugs. If someone is able to pull this branch and test the changes I would be happy to merge this. Just ping me in the PR with the steps you have performed, the tested hardware version(s) and your results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants