Skip to content

Add a clarification for when Security-15-2 is selected to the verification steps of Appearance-26-1 - #130

Draft
yashirot wants to merge 2 commits into
masterfrom
20784_Add-clarification-to-Appearance-26
Draft

Add a clarification for when Security-15-2 is selected to the verification steps of Appearance-26-1#130
yashirot wants to merge 2 commits into
masterfrom
20784_Add-clarification-to-Appearance-26

Conversation

@yashirot

Copy link
Copy Markdown
Contributor

Appearance-26の動作検証手順が「翻訳」メニューの有無を確認するものであるため、
Security-15-2設定時はメニューを選択しても機能しない旨の補足説明を追加することにしました。

@yashirot
yashirot requested a review from piroor August 18, 2026 03:15
Comment thread verify/Appearance Outdated
- アプリケーションメニューの「その他のツール」を開く。
- **確認**
- 「翻訳」という項目がある。
{%if Security_15_2 %}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

すみません説明不足でした。「Firefoxを起動する」から「という項目がある」全部をelse節に入れてしまって、{%if Security_15_2 %}- 必要な通信がSecurity-15-2のカスタマイズにより遮断されており機能が無効化されるため、検証を省略。{%else%}...とする想定でした。
そのようにしていただけますか?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

既存の手順をelse節に入れるように修正しました。

必要な通信がSecurity-15-2のカスタマイズにより遮断されており

Security-15-2を選択しても、lockPref("services.settings.server", "http://localhost");を設定しないケースも実在することから、検証手順自体は省略しない方がよいかなという判断でした。

しかしながらAppearance-26-1は「既定値」の側なので、次年度以降に選択した場合の検証はスキップとなるはずですので、結局は同じと思い直しました。

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確かにそのケースを考慮するとこれだと問題ありでした……
lockPref("services.settings.server", "http://localhost"); を設定する項目を独立して設けるのがいいですかねぇ

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Security-68 として以下を追加する
    Security-68: ブロックリストの取得可否の包括的な制御
        
        :1: 許可する(既定)
        
        -
        
        :1: 禁止する
        
        
        Services.env.set("MOZ_REMOTE_SETTINGS_DEVTOOLS", 1); // https://searchfox.org/mozilla-esr140/rev/564ee8506ed05dfb20fdfb7c4a5609b192b53078/services/settings/Utils.sys.mjs#63
        lockPref("services.settings.server", "http://localhost");
    
  2. Security-15-2からこの2つの設定を削除し、説明文の「上記に加え、以下の設定を行う。」の前に「* ブロックリストの取得を包括的に禁止する(Security-68-2)」を追加する。

として、個別にservices.settings.serverを許可しているケースは「Security-15-2の選択のみ」、そうでない場合でSecurity-15-2を選択しているケースは「Security-15-2とSecurity-68-2を両方とも選択」という設定状況に更新する、というやり方はどうでしょうか?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

方向性として「ブロックリストの取得可否の包括的な制御」を新規の別項目に切り出すのは賛成です。
しかし現段階で設定項目が増えると定義変更のドキュメントに影響が出ますので、その適用は今年度の工期終了後が望ましいです。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ということで、本PRは適用の時期までペンディングにしようと思います。

@yashirot
yashirot requested a review from piroor August 19, 2026 03:00
@yashirot
yashirot force-pushed the 20784_Add-clarification-to-Appearance-26 branch from 842a5c4 to d1c3f6e Compare August 19, 2026 03:05
@yashirot
yashirot marked this pull request as draft August 21, 2026 05:22
@yashirot

Copy link
Copy Markdown
Contributor Author

適用時期が来るまでDraftとします。
10月以降に再開します。

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.

2 participants