{"action":"created","issue":{"url":"https://api.github.com/repos/home-assistant/home-assistant.github.io/issues/4668","repository_url":"https://api.github.com/repos/home-assistant/home-assistant.github.io","labels_url":"https://api.github.com/repos/home-assistant/home-assistant.github.io/issues/4668/labels{/name}","comments_url":"https://api.github.com/repos/home-assistant/home-assistant.github.io/issues/4668/comments","events_url":"https://api.github.com/repos/home-assistant/home-assistant.github.io/issues/4668/events","html_url":"https://github.com/home-assistant/home-assistant.github.io/issues/4668","id":296905981,"number":4668,"title":" Invalid config for mqtt tls_version","user":{"login":"move[bot]","id":33596368,"avatar_url":"https://avatars1.githubusercontent.com/in/6673?v=4","gravatar_id":"","url":"https://api.github.com/users/move%5Bbot%5D","html_url":"https://github.com/apps/move","followers_url":"https://api.github.com/users/move%5Bbot%5D/followers","following_url":"https://api.github.com/users/move%5Bbot%5D/following{/other_user}","gists_url":"https://api.github.com/users/move%5Bbot%5D/gists{/gist_id}","starred_url":"https://api.github.com/users/move%5Bbot%5D/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/move%5Bbot%5D/subscriptions","organizations_url":"https://api.github.com/users/move%5Bbot%5D/orgs","repos_url":"https://api.github.com/users/move%5Bbot%5D/repos","events_url":"https://api.github.com/users/move%5Bbot%5D/events{/privacy}","received_events_url":"https://api.github.com/users/move%5Bbot%5D/received_events","type":"Bot","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-02-13T22:17:30Z","updated_at":"2018-02-13T22:17:32Z","closed_at":null,"author_association":"NONE","body":"*@FFroehlich commented on Feb 13, 2018, 4:38 PM UTC:*\n\nMake sure you are running the latest version of Home Assistant before reporting an issue.\n\nYou should only file an issue if you found a bug. Feature and enhancement requests should go in [the Feature Requests section](https://community.home-assistant.io/c/feature-requests) of our community forum:\n\n**Home Assistant release (`hass --version`):**\n0.63.1\n\n**Python release (`python3 --version`):**\nPython 3.5.3\n\n**Component/platform:**\nRaspberryPi3\n\n**Description of problem:**\nspecification of mqtt tls_version requires specification with apostrophes:\n`tls_version: '1.2'`\nError when using `tls_version: 1.2`:\n\n> [homeassistant.config] Invalid config for [mqtt]: not a valid value for dictionary value @ data['mqtt']['tls_version']. Got 1.2\\. (See /home/homeassistant/.homeassistant/configuration.yaml, line 45). Please check the docs at [https://home-assistant.io/components/mqtt/](https://home-assistant.io/components/mqtt/)\n\n**Expected:**\n`tls_version: 1.2`\n\n**Problem-relevant `configuration.yaml` entries and steps to reproduce:**\n\n```yaml\nmqtt:\n  tls_version: 1.2\n```\n\n1.  `bin/hass --script check_config`\n\n**Traceback (if applicable):**\n\n**Additional info:**\n\n*This issue was moved by @fabaff from home-assistant/home-assistant/issues/12376.*"},"comment":{"url":"https://api.github.com/repos/home-assistant/home-assistant.github.io/issues/comments/365424233","html_url":"https://github.com/home-assistant/home-assistant.github.io/issues/4668#issuecomment-365424233","issue_url":"https://api.github.com/repos/home-assistant/home-assistant.github.io/issues/4668","id":365424233,"user":{"login":"move[bot]","id":33596368,"avatar_url":"https://avatars1.githubusercontent.com/in/6673?v=4","gravatar_id":"","url":"https://api.github.com/users/move%5Bbot%5D","html_url":"https://github.com/apps/move","followers_url":"https://api.github.com/users/move%5Bbot%5D/followers","following_url":"https://api.github.com/users/move%5Bbot%5D/following{/other_user}","gists_url":"https://api.github.com/users/move%5Bbot%5D/gists{/gist_id}","starred_url":"https://api.github.com/users/move%5Bbot%5D/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/move%5Bbot%5D/subscriptions","organizations_url":"https://api.github.com/users/move%5Bbot%5D/orgs","repos_url":"https://api.github.com/users/move%5Bbot%5D/repos","events_url":"https://api.github.com/users/move%5Bbot%5D/events{/privacy}","received_events_url":"https://api.github.com/users/move%5Bbot%5D/received_events","type":"Bot","site_admin":false},"created_at":"2018-02-13T22:17:32Z","updated_at":"2018-02-13T22:17:32Z","author_association":"NONE","body":"*@OttoWinter commented on Feb 13, 2018, 8:51 PM UTC:*\n\nWhen you specify `tls_version: 1.2` the MQTT component _sees_ this as a floating point number, which wouldn't make much sense since version numbers are usually strings (with apostrophes). This is a known limitation/weird behavior of YAML and is usually solved by requiring the user to use quotes. Now, as Home Assistant is often used by people who don't know too much about programming, I would say allowing both versions would also be an option (even if it's not very nice)."}}
{"actor":{"display_login":"move"}}