Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@fishter
Copy link
Contributor

@fishter fishter commented Nov 14, 2025

The login_attempts_threshold parameter is required if ip_ban_enabled is set to true. The default of "-1" for the login_attempts_threshold effectively turns it off. I've updated the sentence about these parameters to make it clearer that both must be set in order to use the functionality.

Proposed change

This is a documentation update which clarifies that an optional parameter (login_attempts_threshold) is required if another optional parameter (ip_ban_enabled) is set to true in the http integration.

Type of change

This clarifies that the login_attempts_threshold is required to be set if ip_ban_enabled is set to true.

  • 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: fixes #

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.

The login_attempts_threshold parameter is required if ip_ban_enabled is set to true. The default of "-1" for the login_attempts_threshold effectively turns it off.  I've updated the sentence about these parameters to make it clearer that both must be set in order to use the functionality.
@fishter fishter requested a review from a team as a code owner November 14, 2025 10:28
@home-assistant home-assistant bot added the current This PR goes into the current branch label Nov 14, 2025
@netlify
Copy link

netlify bot commented Nov 14, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 19e5dd4
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6917045630cc5500088edd83
😎 Deploy Preview https://deploy-preview-41776--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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, @fishter 👍

../Frenck

                       

Blogging my personal ramblings at frenck.dev

@frenck frenck merged commit 85d9ef4 into home-assistant:current Nov 16, 2025
7 checks passed
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.

2 participants