token-exchange-aiac: put the refusal reason in the rejection, not only the message
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 78/100
- issue の種類
- 機能追加
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- go
調査の方向性
core/plugins/tokenexchangeaiac/plugin.go の deny() から始め、次に docs/token-exchange-aiac-plugin.md を読み、拒否応答の形式を確認します。各拒否について安定した理由を details に追加し、その仕様を文書化します。無効化されたターゲットと許可されていないターゲットの両方で、プレーンな403ボディとJSON-RPCの error.data.details の両方に必要な理由が示されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Background
token-exchange-aiac answers two different refusals with the same 403 policy.forbidden:
audience_not_granted: Keycloak has a client for the destination, but the caller is not granted it.target_disabled: the destination's client is disabled, for example a quarantined service.
The reason is recorded in the session event (pctx.Record(... Reason: reason ...)), but the rejection the caller sees carries no Details: deny() in core/plugins/tokenexchangeaiac/plugin.go returns pipeline.DenyStatus(status, code, msg). Only the human-readable message tells them apart ("the destination's client is disabled" vs "this workload is not granted the destination's audience").
rossoctl/aiac's system harness must tell them apart: its quarantine test (test_failed_tool_outbound_call_denied) counts a refusal as "blocked by the quarantine" only for target_disabled. It matches the message text, a copy of the cortex string, which breaks if the message is reworded. Raised in the review of rossoctl/aiac#242.
Task
- Put the machine-stable reason in the rejection, for example
Details: {"reason": "target_disabled"}(andaudience_not_granted,exchange_failed,no_token,foreign_namespace), so it reaches the caller in the plain 403 body (details) and inerror.data.detailsof the JSON-RPC frame. - Document the field in
docs/token-exchange-aiac-plugin.mdas part of the contract.
Acceptance criteria
- A refusal of a disabled target carries
details.reason == "target_disabled"; a refusal of an ungranted target carriesdetails.reason == "audience_not_granted", in both rejection shapes. - The message text is no longer the only way to tell them apart.
Follow-up on the AIAC side: key target_disabled_refusal (k8s/authbridge-rejection.py) on the reason instead of the message.
- 主要言語
- Go
- スター
- 21
- フォーク
- 43
- 平均マージ
- 14時間 25分
- マージ済み PR（30日）
- 245
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
rossoctl/cortex のほかの issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
rossoctl/cortex#1546 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
UI shows costs for all sessions even when drilled into a session対応中かも @esnible が 15 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
rossoctl/cortex#1116 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
bug laptop
難易度 4/5 3〜5日 初心者へのやさしさ 42/100
メンテナーはふだん 1 日以内に返信
-
bug: Copied listener and parser code has drifted (hop-by-hop response headers, SSE BOM/CR handling)オープンbug
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
メンテナーはふだん 1 日以内に返信
-
bug: Ring and ledger count errors and coverage differently, cost reads slow down over time, and cost comments are stale対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンbug
難易度 4/5 3〜5日 初心者へのやさしさ 49/100
メンテナーはふだん 1 日以内に返信
rossoctl/cortex の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
cloudflare/cloudflared#1783 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
orchestration.migration.data.tasks.retry values are ignored by the 8.8 data migration job対応中かも @irisfield が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
camunda/camunda-platform-helm#7773 ·
メンテナーはふだん 1 日以内に返信
-
Add missing unit tests for IsAdmissionPolicyName and IsCertManagerResourceName対応中かも @a1h8 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
kubernetes-sigs/security-profiles-operator#3644 ·
メンテナーはふだん 1 日以内に返信