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

Auth + Hass.io: create system user #15220

Closed
balloob opened this issue Jun 29, 2018 · 2 comments · Fixed by #15536
Closed

Auth + Hass.io: create system user #15220

balloob opened this issue Jun 29, 2018 · 2 comments · Fixed by #15536
Assignees
Labels

Comments

@balloob
Copy link
Member

balloob commented Jun 29, 2018

Hass.io supervisor communicates with Home Assistant via the API. This needs to be adopted for the new auth system.

At startup, the Hass.io component should:

  • create a client. Should be marked as system-generated and cannot be deleted. Should also be marked that it cannot be used for any logins?
  • create a user. Should too be marked as system-generated and cannot be deleted. Should have 0 linked credentials so that it cannot be used to login with.
    -create a refresh token and share it with the supervisor

Should the refresh token have a very long expiration so that Hass.io never needs to refresh? If so, should we just send the access token?

@awarecan
Copy link
Contributor

awarecan commented Jun 30, 2018

We should have scope and issuer property for token/client. In future we can use scope: even more fine grained control access. We can use issuer to decided if user can manage them.

@balloob
Copy link
Member Author

balloob commented Jun 30, 2018

Eventually I want to add those. Don't think that they will be needed to implement system users.

@ghost ghost assigned balloob Jul 18, 2018
@ghost ghost added in progress and removed to do labels Jul 18, 2018
@ghost ghost removed the in progress label Jul 23, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Oct 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants