diff --git a/integration/test_urls.py b/integration/test_urls.py new file mode 100755 index 00000000000..a72de8aa3dd --- /dev/null +++ b/integration/test_urls.py @@ -0,0 +1,67 @@ +"""Integration tests for all urls in Reflex.""" +import os +import re +from pathlib import Path + +import pytest +import requests + + +def check_urls(repo_dir): + """Check that all URLs in the repo are valid and secure. + + Args: + repo_dir: The directory of the repo. + + Returns: + A list of errors. + """ + url_pattern = re.compile(r'http[s]?://reflex\.dev[^\s")]*') + errors = [] + + for root, _dirs, files in os.walk(repo_dir): + if "__pycache__" in root: + continue + + for file_name in files: + if not file_name.endswith(".py") and not file_name.endswith(".md"): + continue + + file_path = os.path.join(root, file_name) + try: + with open(file_path, "r", encoding="utf-8", errors="ignore") as file: + for line in file: + urls = url_pattern.findall(line) + for url in set(urls): + if url.startswith("http://"): + errors.append( + f"Found insecure HTTP URL: {url} in {file_path}" + ) + url = url.strip('"\n') + try: + response = requests.head( + url, allow_redirects=True, timeout=5 + ) + response.raise_for_status() + except requests.RequestException as e: + errors.append( + f"Error accessing URL: {url} in {file_path} | Error: {e}, , Check your path ends with a /" + ) + except Exception as e: + errors.append(f"Error reading file: {file_path} | Error: {e}") + + return errors + + +@pytest.mark.parametrize( + "repo_dir", + [Path(__file__).resolve().parent.parent / "reflex"], +) +def test_find_and_check_urls(repo_dir): + """Test that all URLs in the repo are valid and secure. + + Args: + repo_dir: The directory of the repo. + """ + errors = check_urls(repo_dir) + assert not errors, "\n".join(errors) diff --git a/reflex/.templates/apps/blank/code/blank.py b/reflex/.templates/apps/blank/code/blank.py index 4a1751996e1..8ff44abf19a 100644 --- a/reflex/.templates/apps/blank/code/blank.py +++ b/reflex/.templates/apps/blank/code/blank.py @@ -4,7 +4,7 @@ import reflex as rx -docs_url = "https://reflex.dev/docs/getting-started/introduction" +docs_url = "https://reflex.dev/docs/getting-started/introduction/" filename = f"{config.app_name}/{config.app_name}.py" diff --git a/reflex/.templates/apps/sidebar/README.md b/reflex/.templates/apps/sidebar/README.md index 3d7fa13a9d5..d80145d344a 100644 --- a/reflex/.templates/apps/sidebar/README.md +++ b/reflex/.templates/apps/sidebar/README.md @@ -62,6 +62,8 @@ In this template, we have a sidebar component in `{your_app}/components/sidebar. ### Adding State -As your app grows, we recommend using [substates](https://reflex.dev/docs/substates/overview/) to organize your state. +As your app grows, we recommend using [substates](https://reflex.dev/docs/substates/overview/) +to organize your state. + You can either define substates in their own files, or if the state is specific to a page, you can define it in the page file itself. diff --git a/reflex/app.py b/reflex/app.py index 67bf17f517f..dd8eef7953e 100644 --- a/reflex/app.py +++ b/reflex/app.py @@ -379,7 +379,7 @@ def _generate_component(component: Component | ComponentCallable) -> Component: raise TypeError( "You may be trying to use an invalid Python function on a state var. " "When referencing a var inside your render code, only limited var operations are supported. " - "See the var operation docs here: https://reflex.dev/docs/state/vars/#var-operations" + "See the var operation docs here: https://reflex.dev/docs/vars/var-operations/" ) from e raise e diff --git a/reflex/state.py b/reflex/state.py index 19cb755a05e..055561c169b 100644 --- a/reflex/state.py +++ b/reflex/state.py @@ -278,7 +278,7 @@ def __init__( if not _reflex_internal_init and not is_testing_env(): raise RuntimeError( "State classes should not be instantiated directly in a Reflex app. " - "See https://reflex.dev/docs/state for further information." + "See https://reflex.dev/docs/state/ for further information." ) kwargs["parent_state"] = parent_state super().__init__(*args, **kwargs) diff --git a/reflex/utils/prerequisites.py b/reflex/utils/prerequisites.py index d9fb91c32d3..e5d97671172 100644 --- a/reflex/utils/prerequisites.py +++ b/reflex/utils/prerequisites.py @@ -1036,7 +1036,7 @@ def show_rx_chakra_migration_instructions(): ) console.log("") console.log( - "For more details, please see https://reflex.dev/blog/2024-02-16-reflex-v0.4.0" + "For more details, please see https://reflex.dev/blog/2024-02-16-reflex-v0.4.0/" )