Improve dns_get_record / dns_check_record error handling
まだ誰も着手していません。
- 主要言語
- C
- スター
- 40.4k
- フォーク
- 8.1k
- 平均マージ
- 2日 13時間
- マージ済み PR（30日）
- 96
説明
Description
This is in relation to https://bugs.php.net/bug.php?id=73149 (with 142 up votes) which points problematic error handling of dns_get_record.
Due to BC break, this cannot be handled as a bug fix but there is a space to improve the error handling.
Firstly it would be useful to introduce parameter to switch the error handling to throw exception (e.g. bool $throw), that would allow throwing specialized exception which would allow to identify type of the error.
Additionally it might be also useful to add retry mechanism as some failures for TRY_AGAIN error.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、php-src 内で dns_get_record と dns_check_record の実装とテストを見つけます。リンクされているバグレポートを読み、既存のエラーハンドリング要件を確認したうえで、提案されている例外パラメータ、専用のエラー型、リトライ動作を明確にしてからコードを変更します。合意した API の動作が文書化され、既存の呼び出し元を壊すことなくテストでカバーされれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- c, php
- 領域
- backend-api-design, networking
- issue の種類
- 機能追加
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 25/100