Skip to content

Conversation

cnico
Copy link
Contributor

@cnico cnico commented Mar 13, 2021

Addition of an example with a camera that requires secured access, an HIK camera in my case.

Proposed change

An example of correct secured access : I had to go to the source code to understand how to configure this because still image requires auth parameters and rtsp requires inlined auth.

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

  • Link to parent pull request in the codebase:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue:

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

Addition of an example with a camera that requires secured access, an HIK camera in my case.
@probot-home-assistant probot-home-assistant bot added the current This PR goes into the current branch label Mar 13, 2021
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, @cnico 👍

@frenck frenck merged commit 040aed6 into home-assistant:current Mar 15, 2021
@cnico cnico deleted the patch-2 branch March 15, 2021 20:34
@github-actions github-actions bot locked and limited conversation to collaborators Mar 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-signed current This PR goes into the current branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants