Skip to content

Add call to account status endpoint before attempting to login to fix authentication. #38

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.

Sign up for GitHub

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
bfayers wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Add call to account status endpoint before attempting to login to fix authentication. #38

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
6ffd4de
add account stats GET before login POST
bfayers Feb 12, 2021
b41eadf
set verify=False because dyson run self signed certs
bfayers Feb 12, 2021
ac5566f
use the headers for the status check, maybe unnecesarry; also use jso…
bfayers Feb 12, 2021
8ac02d0
some changes to bring style in line with existing code as well as to …
bfayers Feb 12, 2021
6307f4c
Make changes as suggested by shenxn
bfayers Feb 12, 2021
3965d81
break up the format on line 57
bfayers Feb 13, 2021
f720d24
Updated login tests to work with call to userstatus
bfayers Feb 27, 2021
dfe77cf
Lint the changes for CI
bfayers Feb 27, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion libpurecool/dyson.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ def login(self):
"dyson are using a self signed certificate.")

#Must first check account status
accountstatus = requests.get(f"https://{self._dyson_api_url}/v1/userregistration/userstatus?country={self._country}&email={self._email}")
accountstatus = requests.get(f"https://{self._dyson_api_url}/v1/userregistration/userstatus?country={self._country}&email={self._email}", verify=False)
Copy link
Copy Markdown

@afk11 afk11 Feb 16, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I go to this URL I get cloudflare warning but the SSL certificate is valid. Are they redirecting away from cloudflare or something?

if accountstatus.status_code == requests.codes.ok:
json_status = accountstatus.json()
if json_status['accountStatus'] != "ACTIVE":
Expand Down