PVA/SPVA: track SPVA version independently and move XDG keychain dir pva/<ver> -> spva/<n.n.n>
メンテナーはふだん 2 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
core/pva/src/main/java/org/epics/pva/PVASettings.javaから始めて、バージョン定数とTLSキーチェーンの自動検出パスを確認してください。ハードコードされたバージョンをSPVAのバージョンに変更し、検出にはspva/<version>を使用して、PVXS側の正式なバージョンと値を一致させてください。Javaの検出が新しいディレクトリを使用し、issueで推奨されているように、従来のpva/<version>パスにフォールバックできれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Summary
In the Secure PVAccess (SPVA) line of work, the SPVA release version is being decoupled from the PVXS library version. On the C++ side (PVXS / PVACMS) the SPVA version becomes a separately-tracked number, and the well-known TLS keychain directory changes from pva/<pvxs-version> to spva/<spva-version>.
Phoebus's core/pva/ is an independent Java PVA implementation and does not link the C++ PVXS code, so it cannot consume that version programmatically. It must hard-code the SPVA version (as it already hard-codes the PVA version today) and keep it in sync by hand.
Current behaviour
core/pva/src/main/java/org/epics/pva/PVASettings.java:
// Hard-coded, documented as "Matches PVXS versionString()"
private static final String PVA_VERSION = "1.5";
// Well-known TLS keychain auto-discovery location:
// <xdg_config_home>/pva/<PVA_VERSION>/<filename>
final String path = config_home + File.separator + "pva"
+ File.separator + PVA_VERSION
+ File.separator + filename;
Used to auto-discover server.p12 and client.p12 under ~/.config/pva/1.5/ when EPICS_PVA(S)_TLS_KEYCHAIN is unset, mirroring the PVXS config.cpp fallback.
Requested changes
-
Replace the PVXS-derived
PVA_VERSIONwith a hard-coded SPVA version constant (e.g.SPVA_VERSION), documented as manually mirroring the authoritative PVXS-side SPVA version, with a comment pointing at that source so the two stay in lockstep per SPVA release. -
Change keychain auto-discovery from
pva/<PVA_VERSION>tospva/<SPVA_VERSION>:~/.config/spva/<n.n.n>/server.p12~/.config/spva/<n.n.n>/client.p12
(literal
pva->spva, and the version source changes from PVXS major.minor to the SPVAn.n.n.)
Scope / interop notes
- Java side uses only the config home path here (keychain discovery); there is no data-home / cert-DB usage, unlike the C++ tooling.
- The directory must match whatever PVXS / PVACMS adopt, or the Java client and C++ tools will look in different places for shared keychains. Recommend the same migration behaviour on all sides: fall back to the legacy
pva/<ver>path when the newspva/<n.n.n>one is absent, so existing deployments keep working.
Context
This is the Phoebus-side counterpart of the SPVA version-tracking change being made in the PVXS and PVACMS repositories; those define the authoritative SPVA version and the spva/<n.n.n> convention. Phoebus tracks the same value by hand.
- 主要言語
- Java
- スター
- 130
- フォーク
- 132
- 平均マージ
- 3日 1時間
- マージ済み PR（30日）
- 24
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ControlSystemStudio/phoebus のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
ControlSystemStudio/phoebus#3979 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
ControlSystemStudio/phoebus#3962 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 52/100
ControlSystemStudio/phoebus#3980 ·
メンテナーはふだん 2 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
ControlSystemStudio/phoebus#3978 ·
メンテナーはふだん 2 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
ControlSystemStudio/phoebus#3966 ·
メンテナーはふだん 2 日以内に返信
ControlSystemStudio/phoebus の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
Vakore/ZappierGames#184 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
lugassawan/swe-workbench#823 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
cassiancc/Raspberry-Core#69 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
IronCoreLabs/ironcore-alloy#351 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
checkstyle/checkstyle#22145 ·
メンテナーはふだん 1 日以内に返信