99designs / 99designs/http-signatures-php
Use hash_equals instead of double HMAC approach for signature comparison
- 主要言語
- PHP
- スター
- 46
- フォーク
- 35
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
See https://github.com/99designs/http-signatures-php/pull/28 for what prompted this.
We're currently using a double HMAC approach for signature comparison, as that was the only way for us to securely compare HMAC signatures without making it a breaking change, as the [hash_equals](https://secure.php.net/hash_equals) function we need isn't available until PHP 5.7, and we support PHP 5.5+
When we roll out our next major version we should increase the minimum PHP version to 5.7 or higher, and swap to using `hash_equals`.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
Look for signature comparison code in the codebase, likely in a class handling HMAC verification. Replace the double HMAC comparison with hash_equals. Check the PHP version requirement in composer.json to ensure it's 5.7+. Run existing tests to verify the change doesn't break functionality.
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- php
- 領域
- security
- issue の種類
- リファクタリング
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 45/100