rename UDPProxy to SupportProxy#32
Merged
tridge merged 5 commits intoArduPilot:mainfrom May 9, 2026
Merged
Conversation
Atomic rename of the source file, the built binary, and every reference that has to move with them in the same commit so the build and the docker-test CI job stay green: - git mv udpproxy.cpp supportproxy.cpp - Makefile SOURCES / TARGET / dep rule / help banner / header - docker/Dockerfile + entrypoint.sh paths - .github/workflows/ci.yml docker tag (udpproxy-test → supportproxy-test) and the in-container container name + echo strings - tests/test_config.py: UDPPROXY_BIN → SUPPORTPROXY_BIN, value './udpproxy' → './supportproxy' - tests/conftest.py: import + class default-arg, leaving the UDPProxyProcess class name + log strings for the next commit First step of the rename of the project to SupportProxy. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- tests/conftest.py: class UDPProxyProcess → SupportProxyProcess, per-worker tmpdir prefix udpproxy_w → supportproxy_w, all DEBUG / log strings. - tests/test_connections.py: docstrings + DEBUG strings + the check_udpproxy_output helper → check_supportproxy_output (and every call site), the "udpproxy stdout"/"stderr" labels in the proxy-log dump path. - tests/test_authentication.py + tests/webadmin/test_concurrent.py: docstring/comment. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- scripts/update_server.sh: every ~/UDPProxy/, $HOME/UDPProxy/, and
the udpproxy binary path → SupportProxy / supportproxy. The
pkill / pgrep targets follow.
- start_proxy.sh: pidof, the binary path under $HOME/SupportProxy/,
PYTHONPATH, header comments.
- scripts/setup_ci.sh: header + log line.
- scripts/run_tests.py: docstring + the build/log strings + the
binary-existence check ('udpproxy' → 'supportproxy').
After this commit, scripts/* and start_proxy.sh reference the new
paths only. Server migrations (FireVPS et al.) are still deferred
— update_server.sh will fail against an unmigrated host until the
~/UDPProxy → ~/SupportProxy mv is done out of band.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- webadmin/config.py: WEBUI_TITLE 'UDPProxy admin' → 'SupportProxy admin'. The webui.json override on each server still wins, so this only affects sites that haven't customised the title. - webadmin/__init__.py: module docstring + DEFAULT_GITHUB_REPO_URL pointing at the post-rename ArduPilot/SupportProxy URL. GitHub redirects the old URL post-rename, so this works whether or not the GitHub repo rename has happened yet. - tests/webadmin/test_auth.py: assertion updated to match the new default title. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Final pass: README, source-file docstrings, the architecture diagram, the gitignore / dockerignore binary-name pattern, and the renamed diagram file itself. README keeps one explicit "previously called UDPProxy" callout near the top — that is the only surviving "UDPProxy" reference in the tree. After this commit the acceptance criterion holds: $ git grep -i udpproxy README.md:3: This project was previously called UDPProxy. ... - README.md heading + body + clone URL + docker tags + crontab example. Diagram filename + alt text. - udpproxy-diagram.svg → supportproxy-diagram.svg (file rename) and the title strings inside the diagram. - keydb.py / keydb_lib.py / conntdb.h / conntdb_lib.py / mavlink.h module-header docstrings. - .dockerignore + .gitignore: build-artifact ignore pattern. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
we've moved beyond just UDP quite some time ago