Skip to content

Fix trusted networks deprecated warning message#21538

Merged
balloob merged 1 commit into
home-assistant:devfrom
awarecan:fix-trusted-networks-warning
Feb 28, 2019
Merged

Fix trusted networks deprecated warning message#21538
balloob merged 1 commit into
home-assistant:devfrom
awarecan:fix-trusted-networks-warning

Conversation

@awarecan
Copy link
Copy Markdown
Contributor

@awarecan awarecan commented Feb 28, 2019

Description:

http.trusted_networks has default value, so that the deprecated warning message will always print regardless whether user configured it in yaml.

Related issue (if applicable): fixes beta finding

Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>

Example entry for configuration.yaml (if applicable):

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.

@awarecan awarecan added this to the 0.89.0 milestone Feb 28, 2019
@awarecan awarecan requested a review from a team as a code owner February 28, 2019 17:27
@ghost ghost assigned awarecan Feb 28, 2019
@ghost ghost added the in progress label Feb 28, 2019
@balloob balloob merged commit b18b1cf into home-assistant:dev Feb 28, 2019
@ghost ghost removed the in progress label Feb 28, 2019
@awarecan awarecan deleted the fix-trusted-networks-warning branch February 28, 2019 18:11
balloob pushed a commit that referenced this pull request Mar 1, 2019
@balloob balloob mentioned this pull request Mar 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants