Skip to content

report a switch name that matches no registration - #4

Merged
loicveyssiere merged 1 commit into
mainfrom
v0.3.2
Jul 30, 2026
Merged

report a switch name that matches no registration#4
loicveyssiere merged 1 commit into
mainfrom
v0.3.2

Conversation

@loicveyssiere

Copy link
Copy Markdown
Contributor

Redis accepts any field, so a mistyped switch name was written, reported as applied, and never consulted — indistinguishable from a working pause until someone noticed the flow had never stopped.

App now tells its SwitchBoard which names this process actually checks, and anything else is logged at ERROR. Checked both on set() and on every re-read: set() catches this process's own callers, the re-read catches whatever switches.disable() wrote from outside, which holds a short-lived connection, knows no topology, and so cannot check itself.

Logged rather than raised — a process knows only the names it mounted, and in a split deployment a name may belong to a service running elsewhere. Reported once per name so a 2s refresh cannot turn one typo into a flood.

Also fixes pytest collection: tests/ has no init.py, so pytest put that directory on sys.path instead of the project root and from tests.helpers import ... only resolved under python -m pytest.

Redis accepts any field, so a mistyped switch name was written, reported as
applied, and never consulted — indistinguishable from a working pause until
someone noticed the flow had never stopped.

App now tells its SwitchBoard which names this process actually checks, and
anything else is logged at ERROR. Checked both on set() and on every re-read:
set() catches this process's own callers, the re-read catches whatever
switches.disable() wrote from outside, which holds a short-lived connection,
knows no topology, and so cannot check itself.

Logged rather than raised — a process knows only the names it mounted, and in a
split deployment a name may belong to a service running elsewhere. Reported once
per name so a 2s refresh cannot turn one typo into a flood.

Also fixes pytest collection: tests/ has no __init__.py, so pytest put that
directory on sys.path instead of the project root and `from tests.helpers
import ...` only resolved under `python -m pytest`.

Co-Authored-By: Claude <noreply@anthropic.com>
@loicveyssiere
loicveyssiere merged commit dd71d21 into main Jul 30, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant