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

Clarification on external access for Alexa intents and smart home skills #11555

Merged
merged 1 commit into from
Dec 30, 2019

Conversation

lpomfrey
Copy link
Contributor

Clarifies that your Home Assistant instance must be accessible from the
internet via HTTPS on port 443, using a valid certificate to use account
linking.

Description: Whilst a note about using HTTPS on port 443 was added to the smart home documentation it wasn't added to the custom intent documentation. This adds the note about using HTTPS/443 to the custom intent documentation, and rewords the bit about external access in the requirements section of each to be more specific, following comments on the gist that is linked in the custom intent documentation

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next Home Assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follows the standards.

Clarifies that your Home Assistant instance must be accessible from the
internet via HTTPS on port 443, using a valid certificate to use account
linking.
@probot-home-assistant probot-home-assistant bot added the current This PR goes into the current branch label Dec 28, 2019
@@ -94,8 +94,9 @@ Alexa can link your Amazon account to your Home Assistant account. Therefore Hom
- Find the skill you just created and click `Edit` in the `Actions` column.
- Click `ACCOUNT LINKING` in the left navigation bar of build page
- Input all information required. Assuming your Home Assistant can be accessed by https://[YOUR HOME ASSISTANT URL:PORT]
* `Authorization URI`: https://[YOUR HOME ASSISTANT URL:PORT]/auth/authorize
* `Access Token URI`: https://[YOUR HOME ASSISTANT URL:PORT]/auth/token
* `Authorization URI`: https://[YOUR HOME ASSISTANT URL]/auth/authorize
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does that work if the port is no longer mentioned?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not needed because it has to be the standard HTTPS port (443).

Copy link
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @lpomfrey! 👍

@frenck frenck merged commit 9d0fef4 into home-assistant:current Dec 30, 2019
ochlocracy pushed a commit to ochlocracy/home-assistant.io that referenced this pull request Jan 22, 2020
…-assistant#11555)

Clarifies that your Home Assistant instance must be accessible from the
internet via HTTPS on port 443, using a valid certificate to use account
linking.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
current This PR goes into the current branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants