diff --git a/custom_components/auth_oidc/__init__.py b/custom_components/auth_oidc/__init__.py index 2a604d8..85462b7 100644 --- a/custom_components/auth_oidc/__init__.py +++ b/custom_components/auth_oidc/__init__.py @@ -23,6 +23,8 @@ from .config import ( ROLES, NETWORK, FEATURES_INCLUDE_GROUPS_SCOPE, + FEATURES_DISABLE_FRONTEND_INJECTION, + FEATURES_FORCE_HTTPS, ) # pylint: enable=useless-import-alias diff --git a/custom_components/auth_oidc/config.py b/custom_components/auth_oidc/config.py index 9ab9755..fb0f3cf 100644 --- a/custom_components/auth_oidc/config.py +++ b/custom_components/auth_oidc/config.py @@ -14,6 +14,7 @@ FEATURES_AUTOMATIC_USER_LINKING = "automatic_user_linking" FEATURES_AUTOMATIC_PERSON_CREATION = "automatic_person_creation" FEATURES_DISABLE_PKCE = "disable_rfc7636" FEATURES_INCLUDE_GROUPS_SCOPE = "include_groups_scope" +FEATURES_FORCE_HTTPS = "force_https" CLAIMS = "claims" CLAIMS_DISPLAY_NAME = "display_name" CLAIMS_USERNAME = "username" @@ -69,6 +70,10 @@ CONFIG_SCHEMA = vol.Schema( vol.Optional( FEATURES_INCLUDE_GROUPS_SCOPE, default=True ): vol.Coerce(bool), + # Force HTTPS on all generated URLs (like redirect_uri) + vol.Optional(FEATURES_FORCE_HTTPS, default=False): vol.Coerce( + bool + ), } ), # Determine which specific claims will be used from the id_token diff --git a/custom_components/auth_oidc/endpoints/callback.py b/custom_components/auth_oidc/endpoints/callback.py index e12d196..3ae5ff1 100644 --- a/custom_components/auth_oidc/endpoints/callback.py +++ b/custom_components/auth_oidc/endpoints/callback.py @@ -17,10 +17,14 @@ class OIDCCallbackView(HomeAssistantView): name = "auth:oidc:callback" def __init__( - self, oidc_client: OIDCClient, oidc_provider: OpenIDAuthProvider + self, + oidc_client: OIDCClient, + oidc_provider: OpenIDAuthProvider, + force_https: bool, ) -> None: self.oidc_client = oidc_client self.oidc_provider = oidc_provider + self.force_https = force_https async def get(self, request: web.Request) -> web.Response: """Receive response.""" @@ -38,7 +42,7 @@ class OIDCCallbackView(HomeAssistantView): ) return web.Response(text=view_html, content_type="text/html") - redirect_uri = get_url("/auth/oidc/callback") + redirect_uri = get_url("/auth/oidc/callback", self.force_https) user_details = await self.oidc_client.async_complete_token_flow( redirect_uri, code, state ) @@ -63,4 +67,6 @@ class OIDCCallbackView(HomeAssistantView): return web.Response(text=view_html, content_type="text/html") code = await self.oidc_provider.async_save_user_info(user_details) - return web.HTTPFound(get_url("/auth/oidc/finish?code=" + code)) + return web.HTTPFound( + get_url("/auth/oidc/finish?code=" + code, self.force_https) + ) diff --git a/custom_components/auth_oidc/endpoints/redirect.py b/custom_components/auth_oidc/endpoints/redirect.py index f95fa77..7e59da6 100644 --- a/custom_components/auth_oidc/endpoints/redirect.py +++ b/custom_components/auth_oidc/endpoints/redirect.py @@ -17,13 +17,14 @@ class OIDCRedirectView(HomeAssistantView): url = PATH name = "auth:oidc:redirect" - def __init__(self, oidc_client: OIDCClient) -> None: + def __init__(self, oidc_client: OIDCClient, force_https: bool) -> None: self.oidc_client = oidc_client + self.force_https = force_https async def get(self, _: web.Request) -> web.Response: """Receive response.""" - redirect_uri = get_url("/auth/oidc/callback") + redirect_uri = get_url("/auth/oidc/callback", self.force_https) auth_url = await self.oidc_client.async_get_authorization_url(redirect_uri) if auth_url: diff --git a/custom_components/auth_oidc/helpers.py b/custom_components/auth_oidc/helpers.py index 84d6936..cd7746c 100644 --- a/custom_components/auth_oidc/helpers.py +++ b/custom_components/auth_oidc/helpers.py @@ -4,12 +4,14 @@ from homeassistant.components import http from .views.loader import AsyncTemplateRenderer -def get_url(path: str) -> str: +def get_url(path: str, force_https: bool) -> str: """Returns the requested path appended to the current request base URL.""" if (req := http.current_request.get()) is None: raise RuntimeError("No current request in context") base_uri = str(req.url).split("/auth", 2)[0] + if force_https: + base_uri = base_uri.replace("http://", "https://") return f"{base_uri}{path}" diff --git a/docs/configuration.md b/docs/configuration.md index 4c2e4be..b7e2bff 100644 --- a/docs/configuration.md +++ b/docs/configuration.md @@ -93,6 +93,15 @@ Upon login, OIDC users will then automatically be linked to the HA user with the > [!CAUTION] > MFA is ignored when using this setting, thus bypassing any MFA configuration the user has originally configured, as long as the username is an exact match. This is dangerous if you are not aware of it! +### Forcing HTTPS +First check if you are setting the header `X-Forwarded-Proto` in your proxy and if the [proxy settings for Home Assistant](https://www.home-assistant.io/integrations/http/#use_x_forwarded_for) are configured correctly. You should also check if IP addresses in your logs actually match the origin IP (instead of proxy IP). If you cannot find any mistakes, you may use the following config option to force HTTPS regardless: + +```yaml +auth_oidc: + features: + force_https: true +``` + ### Using a private certificate authority If you use a private certificate authority to secure your OIDC provider, you must configure the root certificates of your private certificate authority. Otherwise you will get an error (`[SSL: CERTIFICATE_VERIFY_FAILED]`) when connecting to the OIDC provider. @@ -132,6 +141,7 @@ Here's a table of all options that you can set: | `features.automatic_person_creation` | `boolean` | No | `true` | Automatically creates a person entry for new user profiles created by this integration. Recommended if you would like to assign presence detection to OIDC users. | | `features.disable_rfc7636` | `boolean`| No | `false` | Disables PKCE (RFC 7636) for OIDC providers that don't support it. You should not need this with most providers. | | `features.include_groups_scope` | `boolean` | No | `true` | Include the 'groups' scope in the OIDC request. Set to `false` to exclude it. | +| `features.force_https` | `boolean` | No | `false` | Set to `true` to force all URLs generated to use `https` instead of automatically determining based on the request scheme or `X-Forwarded-Proto`. | | `claims.display_name` | `string` | No | `name` | The claim to use to obtain the display name. | `claims.username` | `string` | No | `preferred_username` | The claim to use to obtain the username. | `claims.groups` | `string` | No | `groups` | The claim to use to obtain the user's group(s). |