ActivityWatch / ActivityWatch/aw-qt
Server port lookup is Rust-first while the module probe is implementation-specific
- Ngôn ngữ chính
- Python
- Star
- 33
- Fork
- 62
- Merge trung bình
- 18 giờ 54 phút
- Pull request đã merge (30 ngày)
- 5
Mô tả
Split out of the Greptile review on #128 (P1), where it was flagged but is pre-existing on `master` and not profile-specific.
`aw_qt/config.py:_read_server_port()` tries the aw-server-rust config first and falls back to the aw-server (Python) config, then to a default port. `aw_qt/manager.py:Module._get_server_port()` instead reads the config belonging to the module it is probing. When both configs exist and disagree — or when the Python server is the one in `autostart_modules` but only a Rust config sets a port — the tray's dashboard/API links can target a different endpoint than the server the manager actually starts and probes.
Likely fix: resolve the port from whichever server implementation is in `autostart_modules`, rather than a fixed Rust-first order. That changes port resolution for existing installs, so it wants its own PR and its own thought about back-compat.
Noted while adding `--profile` (#128): a custom profile with no port configured falls back to 5600, so a wrong-endpoint link points at the production instance. #128 logs a warning for that case but does not change the lookup order.
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Đánh giá
Issue này chưa được đánh giá.