Add continue statements for both error cases in IP ban loading

Co-authored-by: frenck <195327+frenck@users.noreply.github.com>

homeassistant/components/http/ban.py

@@ -233,6 +233,9 @@ class IpBanManager:
             except vol.Invalid as err:
                 _LOGGER.error("Failed to load IP ban %s: %s", ip_info, err)
                 continue
+            except ValueError:
+                _LOGGER.error("Failed to load IP ban: invalid IP address %s", ip_ban)
+                continue
 
         self.ip_bans_lookup = ip_bans_lookup
 

tests/components/http/test_ban.py

@@ -113,6 +113,48 @@ async def test_access_from_banned_ip_with_partially_broken_yaml_file(
     assert "Failed to load IP ban" in caplog.text
 
 
+async def test_access_from_banned_ip_with_invalid_ip_entry(
+    hass: HomeAssistant,
+    aiohttp_client: ClientSessionGenerator,
+    caplog: pytest.LogCaptureFixture,
+) -> None:
+    """Test that invalid IP addresses in ban file are skipped gracefully.
+
+    An invalid IP entry (e.g., with typo like "Eo128.199.160.243") should
+    be logged as an error and skipped, allowing valid bans to still load.
+    """
+    app = web.Application()
+    app[KEY_HASS] = hass
+    setup_bans(hass, app, 5)
+    set_real_ip = mock_real_ip(app)
+
+    data = {banned_ip: {"banned_at": "2016-11-16T19:20:03"} for banned_ip in BANNED_IPS}
+    # Add invalid IP entries that should be skipped
+    data["Eo128.199.160.243"] = {"banned_at": "2024-07-06T14:07:46"}
+    data["invalidip"] = {"banned_at": "2024-07-06T14:07:46"}
+
+    with patch(
+        "homeassistant.components.http.ban.load_yaml_config_file",
+        return_value=data,
+    ):
+        client = await aiohttp_client(app)
+
+    # Valid banned IPs should still be blocked
+    for remote_addr in BANNED_IPS:
+        set_real_ip(remote_addr)
+        resp = await client.get("/")
+        assert resp.status == HTTPStatus.FORBIDDEN
+
+    # Non-banned IP should have access
+    set_real_ip("192.168.1.1")
+    resp = await client.get("/")
+    assert resp.status == HTTPStatus.NOT_FOUND
+
+    # Check that invalid IP entries were logged
+    assert "Failed to load IP ban" in caplog.text
+    assert "Eo128.199.160.243" in caplog.text
+
+
 async def test_no_ip_bans_file(
     hass: HomeAssistant, aiohttp_client: ClientSessionGenerator
 ) -> None: