-
Notifications
You must be signed in to change notification settings - Fork 17
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
Adds OAuth (Bosch SingleKey ID support) #117
Conversation
Add new mower model
Changes in Bosch API
Add accept_tc_id to variable DEFAULT_BODY for successfull authentication.
Adding more error code consts
Bosch Indego authentication modification
Merged some re-usable code to base client. Added User-Agent header to fix block by Microsoft Azure proxy. Added API call to retrieve the available mowers/serials in the account.
…ent implementation.
Fixed issue where the lib closes the HA HTTP session.
@sander1988 It looks like you got the Indego running again. Thank you, |
Please check this issue: sander1988/Indego#171 This is what we reverse engineered from the Indego Android app. That's what I used to implement this update. |
Thank you, this was helpful. I have this running in Postman now using the Bearer Token. Edit: Works! |
…ion to improve handling by other applications like HomeAssistant.
I just added a small update to version 2.2.0 . This adds the option raise_request_exceptions. This way we can detect these kind of errors in the applications (like HA) make some decisions based on it. For example to delay the next update, to prevent update loops. I have seen this issues where a 502 on the Bosch side caused the client to do > 10 requests a second. |
Same here: would be great to do pull requests to the develop branch. I always do this when i develop new features or trying to fix bugs. |
You can the PR right? As you are the project owner. |
I dont know how. |
Ok, I have changed it. |
Note: I have only tested the calls used by the HA integration (https://github.com/jm-73/Indego/) .