aws / aws/aws-encryption-sdk-java
Allow to pass overrideConfiguration per request
- 主要言語
- Java
- スター
- 240
- フォーク
- 125
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
### Security issue notifications
If you discover a potential security issue in the AWS Encryption SDK we ask that you notify AWS Security via our [vulnerability reporting page](https://aws.amazon.com/security/vulnerability-reporting/). Please do **not** create a public GitHub issue.
### Problem:
If i want to pass additional headers with Decrypt call (e.g Confused Deputy protection). I need to provide for each account own KmsClient. Instead in SdkV2 i can call .overrideConfiguration on request (you already use it for API_NAMESPACE).
If it will be possible to add .overrideConfiguration per call encrypt/decrypt i can use the same KmsClient for all accounts.
### Solution:
A description of the possible solution in terms of Encryption SDK architecture.
I see 2 options:
1. in AwsCrypto.decryptData/ecryptData provide additional argument with options
2. When construct KmsMasterKeyProvider ask for supplier for override. But there we need to send something in addition to identify context of request
### Out of scope:
Is there anything the solution will intentionally NOT address?
[//]: # (NOTE: If you believe this might be a security issue, please email aws-security@amazon.com instead of creating a GitHub issue. For more details, see the AWS Vulnerability Reporting Guide: https://aws.amazon.com/security/vulnerability-reporting/ )
コントリビューションガイド
調査の方向性
まず AwsCrypto.decryptData/encryptData と KmsMasterKeyProvider を追跡し、リクエストごとの AWS SDK v2 overrideConfiguration をどこに組み込めるかを理解します。既存の API_NAMESPACE の使用方法を 2 つの提案されたオプションと比較し、そのうえで encrypt と decrypt に必要なリクエストコンテキストと API の形を定義します。完了の条件は、1 つの KmsClient で複数のアカウントからのリクエストを処理でき、追加のヘッダーが KMS 呼び出しに到達することです。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- aws, java
- 領域
- backend-api-design, security
- issue の種類
- 機能追加
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 25/100