Skip to content

Remove stale user salts#19004

Merged
balloob merged 1 commit intohome-assistant:devfrom
Eriner:small-auth-cleanup
Dec 4, 2018
Merged

Remove stale user salts#19004
balloob merged 1 commit intohome-assistant:devfrom
Eriner:small-auth-cleanup

Conversation

@Eriner
Copy link
Copy Markdown
Contributor

@Eriner Eriner commented Dec 4, 2018

Description:

data['salt'] was originally used as a part of the pbkdf2 implementation.
I failed to remove this as a part of the cleanup in #18736.

Example entry for configuration.yaml (if applicable):

auth_providers:
  - type: homeassistant

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.

user['salt'] was originally used as a part of the pbkdf2 implementation.
I failed to remove this as a part of the cleanup in home-assistant#18736.
@homeassistant homeassistant added cla-signed core small-pr PRs with less than 30 lines. labels Dec 4, 2018
@ghost ghost added the in progress label Dec 4, 2018
@balloob balloob merged commit 38b09b1 into home-assistant:dev Dec 4, 2018
@ghost ghost removed the in progress label Dec 4, 2018
@balloob balloob mentioned this pull request Dec 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed core small-pr PRs with less than 30 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants