Skip to content
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

Support For Federated Login #87

Open
3 tasks done
mathias-ewald opened this issue Oct 21, 2023 · 4 comments
Open
3 tasks done

Support For Federated Login #87

mathias-ewald opened this issue Oct 21, 2023 · 4 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@mathias-ewald
Copy link

Checklist

  • I have filled out the template to the best of my ability.
  • This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
  • This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.

Problem: I cannot use the integration with my Ambrogio account. Login does not work.

Describe the solution you'd like

Ambrogio allows to login via Google and Apple accounts. While setting up the integration, I am asked to provide credentials (I only assume that, as the form does not give any hints about what's expected in the textfields). I would like to have the option here to login with my Google account.

Screenshot 2023-10-21 at 11 28 14

Describe alternatives you've considered

I could probably delete my Robot and all other devices from my account, create a non-frederated account with Ambrogio and readd everything. But do I want that? Identify federation is a best practice anyway and way more secure due to potential support of MFA.

Additional context

None

@sHedC
Copy link
Owner

sHedC commented Oct 23, 2023

Hi

Thanks for this, so no you don't have to do what you said, I am sorry not had time to work on this at all but want to. ZCS Lawn Mower Robot add-on works better than this right now.

The Ambrogio login is mainly a front to picking up the robot details to log in directly. The ZCS uses things like the IMEI.

I will start working on this again.

@sHedC sHedC self-assigned this Oct 23, 2023
@sHedC sHedC added the enhancement New feature or request label Nov 12, 2023
@sHedC sHedC added this to the Release 0.0.1 milestone Nov 12, 2023
@ufozone
Copy link
Contributor

ufozone commented Feb 24, 2024

@sHedC, nice work! The login and getting the garage details looks great.

My integration has many features in the entities and services. Your integration has the much better and intuitive setup. I would be happy if we combine our development capacity.

@sHedC
Copy link
Owner

sHedC commented Feb 26, 2024

@sHedC, nice work! The login and getting the garage details looks great.

My integration has many features in the entities and services. Your integration has the much better and intuitive setup. I would be happy if we combine our development capacity.

Hi ufozone should would however if you are talking about the federated login it only works if home assistant is running on localhost, the auth is restricted to only 3 domains one is localhost so always gets rejected once on the server :( Not worked out how to deal with that.

@ufozone
Copy link
Contributor

ufozone commented Mar 1, 2024

Argh, of course I briefly tested the latest version in my local dev environment. The source code also looked promising. :-/
I don't find entering the client key so good in my integration. In the iPhone app, for example, it is not easy to read.

We'll have to wait and see anyway: I got the information (from a forum, Facebook group or something) that a completely new app will be released in spring 2024.

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

When branches are created from issues, their pull requests are automatically linked.

3 participants