Relay rejects host names, so a changing iMotions IP can't be entered as a name
メンテナーはふだん 4 日以内に返信
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 48/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- rust
- 領域
- networking
調査の方向性
Start with State::validate_tcp_addr in relay/src/state.rs and check how its result is used when connecting; the issue proposes resolving host:port with ToSocketAddrs. Confirm the client's network need first, since the issue says it is unverified and asks for triage. Done means host names and IPs are accepted, while unresolved names and missing or out-of-range ports produce clear errors.
索引モデルが issue の本文から書いたものです。
説明
Is your feature request related to a problem? Please describe.
State::validate_tcp_addr (relay/src/state.rs) parses the address as a SocketAddr, so only a literal IP and port are accepted. localhost:9999 and a machine name such as IMOTIONS-PC:<port> are rejected. The README documents this ("localhost:9999 is rejected"), but it may be a problem in the client's lab. Her simulator and iMotions run on different machines (the plugin runs on the simulator machine, iMotions on its own computer), so the relay needs the iMotions computer's network address. If that address comes from DHCP it can change between sessions, and the saved-address dropdown would then hold a stale entry.
Describe the solution you'd like
Accept host:port where host is a name or an IP, resolving it at connect time (ToSocketAddrs), and keep a clear error for a name that does not resolve or a port that is missing or out of range.
Describe alternatives you've considered
Keep literal IPs only and ask the lab to give the iMotions machine a fixed IP. That costs nothing in code but needs the lab's IT. It is worth asking the client before building this.
Additional context
Not confirmed as a real need: we don't know how the lab's network is set up, so this is needs-triage until the client answers whether the iMotions machine has a fixed IP. Related: #195 (which port iMotions listens on) and #139 (saved addresses, closed).
- 主要言語
- Python
- スター
- 4
- フォーク
- 8
- 平均マージ
- 5日 7時間
- マージ済み PR（30日）
- 11
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
oss-slu/PilotDataSynchronization のほかの issue
-
Connect TCP with an invalid address is silent in the event log対応中かも @PRINCEKK122 が 10 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
oss-slu/PilotDataSynchronization#241 ·
メンテナーはふだん 4 日以内に返信
-
area:ml bug python
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
oss-slu/PilotDataSynchronization#232 ·
メンテナーはふだん 4 日以内に返信
-
area:docs area:xplane-plugin documentation ready-for-human
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
oss-slu/PilotDataSynchronization#194 · コメント 1 件 ·
メンテナーはふだん 4 日以内に返信
-
No CI builds or tests the Rust crates対応中かも @PRINCEKK122 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 25/100
oss-slu/PilotDataSynchronization#252 ·
メンテナーはふだん 4 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 63/100
oss-slu/PilotDataSynchronization#251 ·
メンテナーはふだん 4 日以内に返信
oss-slu/PilotDataSynchronization の issue をすべて見る
似ている issue
-
area/profiles comp/cli P3 type/feature
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
NousResearch/hermes-agent#146372 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信
-
[oblt-aw][autodoc] Document obs-aw-automerge contract and fix automerge doc inconsistencies対応中かも @github-actions が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
GeoSeries scale and skew tests fail with pytest 9.2.0対応中かも @jiayuasu が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
admission-deferred priority: medium status: ready work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
melodic-software/claude-code-plugins#10887 · コメント 4 件 ·
メンテナーはふだん 1 日以内に返信